1558485 - Enable some ESLint rules for the rest of dom/

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task)

Description

[Mark Banner (:standard8)]

As part of rolling out ESLint across the tree, I'm changing the approach slightly.

I'm looking to get ESLint running on all of the tree with whatever rules we can enabled, rather than getting ESLint running on new sections with all rules enabled.

This will get us at least basic syntax coverage, make it easier to roll out new rules across the tree, and make it simpler to enable the currently failing rules - we can enable them in small blocks, rather than big lumps.

I'm currently working on a patch for this, just figuring out the last few issues.

Comment 1

[Mark Banner (:standard8)]

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=37e133f8b3b6d4f621e44d7d15383d5849360015

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1558485 - Turn on ESLint for all of dom/ disabling most of the failing rules. r?smaug

Depends on D34760

Comment 3

[Mark Banner (:standard8)]

Updated try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=c6f13a59268db6bbc617eaf8a9c5c94982d80184&selectedJob=251694430

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b3a69dff7e25
Turn on ESLint for all of dom/ disabling most of the failing rules. r=smaug

Comment 5

[Daniel Varga [:dvarga]]

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