Harmonise the ESLint configuration for eslint-plugin-mozilla/spidermonkey-js with the Mozilla recommended one

RESOLVED FIXED in Firefox 55

Status

enhancement
RESOLVED FIXED
2 years ago
8 months ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

3 Branch
mozilla55

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(3 attachments)

It gets slightly annoying that there's a few differences between the Mozilla recommended ESLint config, and the one we currently use for our eslint-plugin-*.

Hence, I'd like to unify them a bit and reduce the duplicate rules.
Comment on attachment 8857432 [details]
Bug 1355778 - Autofix ESLint issues in eslint-plugin-* due to the rule changes.

https://reviewboard.mozilla.org/r/129430/#review132124
Attachment #8857432 - Flags: review?(dtownsend) → review+
Comment on attachment 8857433 [details]
Bug 1355778 - Manual fix of ESLint issues in eslint-plugin-* due to the rule changes.

https://reviewboard.mozilla.org/r/129432/#review132128
Attachment #8857433 - Flags: review?(dtownsend) → review+
Comment on attachment 8857431 [details]
Bug 1355778 - Unify some of the eslint-plugin-* rules to match the Mozilla recommended configuration.

https://reviewboard.mozilla.org/r/129428/#review132130
Attachment #8857431 - Flags: review?(dtownsend) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0baa1e8edbd7
Unify some of the eslint-plugin-* rules to match the Mozilla recommended configuration. r=mossop
https://hg.mozilla.org/integration/autoland/rev/3898b0c456d6
Autofix ESLint issues in eslint-plugin-* due to the rule changes. r=mossop
https://hg.mozilla.org/integration/autoland/rev/cf2b51a98a3f
Manual fix of ESLint issues in eslint-plugin-* due to the rule changes. r=mossop
Product: Testing → Firefox Build System
Version: Version 3 → 3 Branch
You need to log in before you can comment on or make changes to this bug.