Closed
Bug 1231957
(devtools-eslint)
Opened 9 years ago
Closed 4 years ago
[meta] Fix devtools files so that they can be removed from .eslintignore
Categories
(DevTools :: General, defect)
DevTools
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: miker, Unassigned, Mentored)
References
(Blocks 1 open bug)
Details
(Keywords: meta)
We need to go through .eslintignore and fix all eslint warnings and errors in devtools files.
Reporter | ||
Updated•9 years ago
|
Whiteboard: [good-first-bug lang=js]
Comment 1•9 years ago
|
||
I'd argue that this should be a meta bug and that we create smaller bugs for each file we clean up. I also think we should wait for bug 1229106 to be fixed first. There's no sense in removing files from .eslintignore until eslint runs automatically somewhere, otherwise new errors will creep in without us realizing.
Keywords: meta
Summary: Fix devtools files so that they can be removed from .eslintignore → [meta] Fix devtools files so that they can be removed from .eslintignore
Removing GFB here, since it should likely be handled in the individual children if desired.
Whiteboard: [good-first-bug lang=js]
Updated•8 years ago
|
Alias: devtools-eslint
Depends on: 1278357
Depends on: 1278413
Depends on: 1283583
Depends on: 1283886
Updated•7 years ago
|
Depends on: dt-eslint-rules
Depends on: 1349009
Updated•7 years ago
|
Blocks: devtools-debtools
Updated•7 years ago
|
Blocks: dt-polish-debt
Updated•7 years ago
|
No longer blocks: devtools-debtools
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•5 years ago
|
No longer depends on: dt-eslint-rules
Comment 5•4 years ago
|
||
As far as I can tell, we've now removed all the devtools entries from .eslintignore
that we can. The remaining items are generated code, sourcemaps, intentionally broken code etc.
The only part remaining is the web replay code which is being removed in bug 1609815.
However, I think we can mark this one as fixed now and track that there.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•