1248161 - Remove duplicated entry in devtools/.eslintrc

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

devtools/.eslintrc contians duplicated entry

"space-after-function-name": 0,

Honza

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Attachment: bug1248161.patch

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/f50b9deac136

Comment 3

[Patrick Brosset <:pbro>]

I've pushed this to fx-team after realizing that moving to eslint 2.0 broke our eslint build ...
If you run mach eslint --setup today, this upgrades eslint to 2.0.0 (which must have been released on npm last week), and because it comes with a number of non-backward compatible changes, that ends up breaking some things.

It also emits 2 warnings for other rules. I'll file another bug.
warning  Rule 'space-after-keywords' was removed and replaced by: keyword-spacing     space-after-keywords
warning  Rule 'space-return-throw-case' was removed and replaced by: keyword-spacing  space-return-throw-case

Comment 4

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Great, thanks!
Honza

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f50b9deac136