Closed
Bug 1558485
Opened 5 years ago
Closed 5 years ago
Enable some ESLint rules for the rest of dom/
Categories
(Core :: DOM: Core & HTML, task)
Core
DOM: Core & HTML
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
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.
|
Assignee | |
Updated•5 years ago
|
Points: --- → 2
|
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
Depends on D34760
|
|
Assignee | |
Comment 3•5 years ago
|
||
Pushed by mbanner@mozilla.com:
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•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
|
|
Assignee | |
Updated•5 years ago
|
Iteration: --- → 69.3 - Jun 10 - 23
You need to log in
before you can comment on or make changes to this bug.
Description
•