1782819 - Remove unnecessarily turned off ESLint rules and enable a few for devtools

Status: RESOLVED FIXED

(DevTools :: General, task)

Description

[Mark Banner (:standard8)]

In devtools/.eslintrc.js there are a lot of rules that are turned off, and can be removed from the configuration as those rules are off by default. This will help to establish a clearer picture about what is different in the devtools configuration to the rest of mozilla-central.

Additionally, where some of those rules are turned off, they are only failing a few instances and so I think we should turn them on to again reduce the differences to mozilla-central.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1782819 - Remove ESlint rules from devtools/.eslintrc.js where they are already turned off or obsolete. r?jdescottes

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1782819 - Enable some ESLint rules for devtools to be consistent with the rest of mozilla-central. r?jdescottes

Depends on D153536

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f45e0fa02ef
Remove ESlint rules from devtools/.eslintrc.js where they are already turned off or obsolete. r=jdescottes
https://hg.mozilla.org/integration/autoland/rev/bfbb483ddbfe
Enable some ESLint rules for devtools to be consistent with the rest of mozilla-central. r=jdescottes

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/6f45e0fa02ef
https://hg.mozilla.org/mozilla-central/rev/bfbb483ddbfe