This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 482505 - [eslint] Update to ESLint 1.9.0
Summary: [eslint] Update to ESLint 1.9.0
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: 11.0   Edit
Hardware: PC Mac OS X
: P3 enhancement (vote)
Target Milestone: 11.0   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-18 13:06 EST by Michael Rennie CLA
Modified: 2015-11-26 13:49 EST (History)
4 users (show)

See Also:


Attachments
Proposed patch (98.34 KB, patch)
2015-11-26 10:14 EST, Olivier Thomann CLA
no flags Details | Diff
Proposed patch (108.74 KB, patch)
2015-11-26 10:31 EST, Olivier Thomann CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2015-11-18 13:06:21 EST
It has been a while since we updated ESLint. The library is now at version 1.9.0.

There are many updates we can make use of:

1. plugins - we could get rid of our edits to the lib and push all our 'orion addons' into a plugin
2. new API that can take infos from a AST (we no longer have to make changes to the API to use it)
3. fixes - rules can now provide the code to use to fix the problem reported - we might be able to hook this up to our quick fixes to provide 'fix all problems of this type' and get rid of the command-based quick fixes.

The CQ is here: https://dev.eclipse.org/ipzilla/show_bug.cgi?id=10397

I also asked if we could re-examine using the stock rule set from eslint.
Comment 1 Carolyn MacLeod CLA 2015-11-18 13:43:49 EST
Not sure whether a re-read of Bug 467903 is useful.
Comment 2 Michael Rennie CLA 2015-11-18 14:49:11 EST
ESLint now makes use of the 'globals' package (really just a JSON file), I have opened a CQ for that as well:

https://dev.eclipse.org/ipzilla/show_bug.cgi?id=10400
Comment 3 Olivier Thomann CLA 2015-11-26 10:02:58 EST
I'll provide a patch for this one.
Comment 4 Olivier Thomann CLA 2015-11-26 10:03:55 EST
I will adopt the new globals version at the same time. See https://dev.eclipse.org/ipzilla/show_bug.cgi?id=10400
Comment 5 Olivier Thomann CLA 2015-11-26 10:14:33 EST
Created attachment 258289 [details]
Proposed patch

This patch also includes the new rule for no-mixes-spaces-and-tabs.
Comment 6 Olivier Thomann CLA 2015-11-26 10:31:03 EST
Created attachment 258290 [details]
Proposed patch

New version with html file updated.
Comment 7 Michael Rennie CLA 2015-11-26 13:49:48 EST
Merged with:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=d0bf46fae8044c58037329b8f77765a4faad51cf

All tests pass.

With this commit we are not directly consuming the rules from ESLint. We can do that work in bug 467903