Commit Graph

14 Commits

Author SHA1 Message Date
Kris Maglione
1fb8f15cd8 Bug 1229874: Part 3 - Enable ESLint in WebExtension code. r=billm
The base .eslintrc is essentially a merge of the root Toolkit .eslintrc and
the devtools .eslintrc, with some minor changes to match our prevalent style.

For the most enforces the coding styles that we've been using most
consistently. There are a couple of significant differences, though:

 * The rule for opening brace alignment can only be applied globally, and
   doesn't make exceptions for top-level functions. I chose to turn it on, and
   change the brace style of existing top-level functions that violated it,
   since the rule seemed worth using, and that's the direction most Toolkit JS
   code has been headed anyway.

 * The rule for switch/case statements requires an added indentation level for
   case statements. Most of our switch statements did not use an extra level
   of indentation, and I initially wrote the rule to enforce that style, until
   I came across case statements that used blocks, and required the extra
   indentation level for sanity.
2015-12-02 16:58:53 -08:00
Dave Townsend
625465b938 Bug 1229519: Fix miscellaneous parts of toolkit to pass eslint checks. r=MattN 2015-12-03 10:02:45 -08:00
Dave Townsend
d2de4624de Backed out 5161ded671e0 for causing Mulet failures 2015-12-03 20:01:43 -08:00
Dave Townsend
60c87c661d Bug 1229519: Fix miscellaneous parts of toolkit to pass eslint checks. r=MattN 2015-12-03 10:02:45 -08:00
Wes Kocher
4aa50ddd49 Merge m-c to fx-team, a=merge 2015-12-03 16:12:01 -08:00
Fabrice Desré
3dfc2cff96 Bug 1229620 - Replace #ifdefs in b2g/ code by AppConstants r=timdream 2015-12-03 09:14:07 -08:00
Robert Helmer
c5ca575245 bug 1228792 - remove addons manager from eslint ignore r=mossop 2015-12-02 18:41:58 -08:00
Felipe Gomes
26bf843a09 Bug 1229603 - Make most of browser/base lintable by removing non-standard syntax. r=Mossop 2015-12-02 17:51:26 -02:00
Michael Comella
27fa345bb7 Bug 1229433 - Ignore HomeProvider in eslint. r=nalexander 2015-12-01 10:06:52 -08:00
Mark Banner
e235d2f350 Bug 1229004 - Rename the libs directory to vendor in browser/extensions/loop/standalone/content. r=dmose 2015-12-01 14:47:50 +00:00
Mark Banner
a2902cbee3 Bug 1228628 - Add a minimal .eslintrc configuration for browser and start linting a few browser files with basic rules. r=Mossop 2015-11-30 11:42:25 +00:00
Phil Ringnalda
19851ebadb Back out 343223ce6b34 (bug 1228628) for Experiments xpcshell bustage
CLOSED TREE
2015-11-27 20:55:17 -08:00
Mark Banner
70f59040f6 Bug 1228628 - Add a minimal .eslintrc configuration for browser and start linting a few browser files with basic rules. r=Mossop 2015-11-27 22:37:38 +00:00
Dave Townsend
67b6efe88d Bug 1228628: Move all eslintignore rules to the top-level of the tree. r=mconley 2015-11-27 09:39:59 -08:00