Commit Graph

21 Commits

Author SHA1 Message Date
Kris Maglione
8d948cbdfc Bug 1230776: Support scripts from HTML files in eslint. r=gps
Also removes related unused variables in mach_commands.py.

--HG--
extra : commitid : IiDVMuEZtA5
extra : rebase_source : 575a51dd0ad5450323b4da5f441f8e5d721e41d6
2015-12-05 13:17:49 -08:00
Gijs Kruitbosch
c652fdb86e Bug 1233492 - make browser/experiments eslintable, r=felipe
--HG--
extra : commitid : AlqcGlqXwrj
extra : rebase_source : 2daaa302b04cef72b173265dccdba55a538e58eb
2015-12-17 19:42:24 +00:00
Gijs Kruitbosch
2a29a25860 Bug 1233470 - make browser/modules eslintable, r=Mossop
--HG--
extra : commitid : IZkQYhmLl9V
extra : rebase_source : 82bd0ac1e9f3b3267d60d127758fa5771a5e9fc9
extra : amend_source : aa17ba3c05cd56840df3c986ba04221beeac2026
2015-12-17 18:31:08 +00:00
Dave Townsend
92dfb90a5f Back out the videocontrols piece of bug 1231828 due to b2g breakage and add to the ignore list. r=backout
--HG--
extra : commitid : 82kdyIUHBLg
extra : rebase_source : 0fb71c46271e8616473c751d19845e13df5e7eea
2015-12-16 12:45:58 -08:00
Patrick Brosset
b9206ee742 Bug 1229859 - Massively reduce the number of eslint errors in devtools by ignoring lib files, adding missing .eslintrc files and making some rules be warnings; r=Mossop
--HG--
extra : commitid : F4M9s89to3Y
extra : rebase_source : 768a99038b43f548d8e0f88248d1be1fe7e5c579
extra : histedit_source : 619e1e7f909a09a7f5731ad8cef9e9a714d76b5e%2Ca78d84e263764af29e9e9e42f419f8189b223d8c
2015-12-10 15:46:32 -05:00
Robert Helmer
865238c5a1 Bug 1230861 - remove add-ons manager from ESLint ignore file. r=mossop
--HG--
extra : transplant_source : B%C5/%B6%0E5H%89n%B2%CA%5E%9F%F7%0A%40_%11%C8%7C
2015-12-06 17:59:36 -08:00
Marco Bonardo
1006b4755f Bug 1230549 - Storage should pass more eslint rules. r=yoric
--HG--
extra : commitid : 9FE8bR3AhsJ
extra : rebase_source : 1d65cc419e9375facb1dc7265e05ee84195153e7
2015-12-04 17:00:03 +01:00
Kris Maglione
b729a30707 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.

--HG--
rename : devtools/.eslintrc => toolkit/components/extensions/.eslintrc
extra : commitid : 4aDZOi5MNPc
extra : rebase_source : f34ba43aeb8046a0c971cb76b76410497bd12b42
extra : source : 2a7eb4ef7b375e5fd52d843ca206a071367a6475
2015-12-02 16:58:53 -08:00
Dave Townsend
6fcdc1e67d Bug 1229519: Fix miscellaneous parts of toolkit to pass eslint checks. r=MattN
--HG--
extra : commitid : 1iLsb7ZpSpI
extra : rebase_source : ed920934f04e520f5410454ec3930cd5992aa965
2015-12-03 10:02:45 -08:00
Dave Townsend
3f712c761e Backed out 5161ded671e0 for causing Mulet failures
--HG--
extra : commitid : AcnOYAhDWhq
extra : rebase_source : 74bdf37c69b2518534c77ea0d29bfbe4f1166f62
2015-12-03 20:01:43 -08:00
Dave Townsend
6bafb02444 Bug 1229519: Fix miscellaneous parts of toolkit to pass eslint checks. r=MattN
--HG--
extra : commitid : HgkrvB0uHPG
extra : rebase_source : f3ac923e3d621d45b70951fc33dccc68a07c228d
2015-12-03 10:02:45 -08:00
Wes Kocher
4582cabbf5 Merge m-c to fx-team, a=merge
--HG--
extra : commitid : DhYTrQryYUJ
2015-12-03 16:12:01 -08:00
Fabrice Desré
73cba43755 Bug 1229620 - Replace #ifdefs in b2g/ code by AppConstants r=timdream 2015-12-03 09:14:07 -08:00
Robert Helmer
29efc6f82d bug 1228792 - remove addons manager from eslint ignore r=mossop
--HG--
extra : commitid : KdM9mSmsxcJ
extra : rebase_source : a856987a05a67ca02dafa1a73c6c64072503262e
extra : histedit_source : 2134b25f630d9fe9976c7c81891b12525bf3044d
2015-12-02 18:41:58 -08:00
Felipe Gomes
2df6ae44b6 Bug 1229603 - Make most of browser/base lintable by removing non-standard syntax. r=Mossop
--HG--
extra : commitid : 5Dg6Q0MzXTN
2015-12-02 17:51:26 -02:00
Michael Comella
a32de0940d Bug 1229433 - Ignore HomeProvider in eslint. r=nalexander
--HG--
extra : commitid : BU03tmoAA6r
extra : rebase_source : 95655b1fb084ec2b5206445be6eca56e9c9fe7d6
2015-12-01 10:06:52 -08:00
Mark Banner
c3c17919f1 Bug 1229004 - Rename the libs directory to vendor in browser/extensions/loop/standalone/content. r=dmose
--HG--
rename : browser/extensions/loop/standalone/content/libs/l10n-gaia-02ca67948fe8.js => browser/extensions/loop/standalone/content/vendor/l10n-gaia-02ca67948fe8.js
rename : browser/extensions/loop/standalone/content/libs/l10n-gaia-upstream.txt => browser/extensions/loop/standalone/content/vendor/l10n-gaia-upstream.txt
2015-12-01 14:47:50 +00:00
Mark Banner
c96e025e76 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
82cfbc8176 Back out 343223ce6b34 (bug 1228628) for Experiments xpcshell bustage
CLOSED TREE
2015-11-27 20:55:17 -08:00
Mark Banner
e8e5bab529 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
3593419254 Bug 1228628: Move all eslintignore rules to the top-level of the tree. r=mconley
--HG--
extra : commitid : Dwq9c2G0xAV
extra : rebase_source : 29ef8c7dad0b0477933d82ea87fd157607cc7b14
extra : amend_source : f0353d9eab87d940d29377401b60606153644985
2015-11-27 09:39:59 -08:00