Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 496738

Summary: .eslintrc.json is not working for Brackets source
Product: [ECD] Orion Reporter: Steve Northover <steve_northover>
Component: JS ToolsAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED WORKSFORME QA Contact:
Severity: normal    
Priority: P2 CC: curtis.windatt.public, Olivier_Thomann
Version: 12.0   
Target Milestone: 13.0   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Works for me. none

Description Steve Northover CLA 2016-06-24 10:42:52 EDT 
1) get the source for Brackets
2) click on Gruntfile.js
3) linting is not running (see errors at the top of the file)
Comment 1 Michael Rennie CLA 2016-06-24 12:07:52 EDT 
Linting is indeed running - the errors at the top are just letting you know there are rules defined in the file that don't exist.

To confirm all is well, go to line 351 and remove the semicolon -> you immediately get a lint error. Their file turns off most of the linting rules (except for semicolons and a handful of others).

I'm going to close this worksforme - I don't think we want to hide bad / unknown lint rules and linting is working as expected.
Comment 2 Steve Northover CLA 2016-06-24 12:21:58 EDT 
I type:

fred = 12;

in a file and get no error that I defined a new bogus global.
Comment 3 Steve Northover CLA 2016-06-24 12:23:32 EDT 
I need to save the file to see the error.
Comment 4 Olivier Thomann CLA 2016-06-24 12:25:38 EDT 
Created attachment 262683 [details]
Works for me.
Comment 5 Steve Northover CLA 2016-07-14 20:07:46 EDT 
Sorry about that!  I should have closed this as works for me after making comment #3.  I was using Orion electron and you need to save the file to see the error.