.eslintignore no longer needs to ignore directories with no js files

RESOLVED FIXED in Firefox 69

Status

task
P1
normal
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
mozilla69
Dependency tree / graph

Firefox Tracking Flags

(firefox69 fixed)

Details

Attachments

(1 attachment)

When we first added ESLint, we added a few directories to the ignore list because it helped ESLint's performance.

I believe at some stage along the way, ESLint improved their indexing performance. Regardless, "including" these directories no longer gives any noticeable performance hit on my machine, so I think we should just include these.

It also will help prevent files sneaking into those directories that should be linted.

Due to these removals, ESLint is now enabled for build/pgo/index.html, so I'm adding a few extra fix ups to make that pass. I've not changed the formatting, since that will hopefully be done with prettier soon.

I'm also pulling the dom lines to the top of the file, since these are the only remaining general exclusions that we're still working on.

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0d2958471c08
eslintignore no longer needs to ignore directories with no js files. r=mossop
Backout by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/75d012801085
Backed out changeset 0d2958471c08 for spidermonkey bustages. CLOSED TREE
Pushed by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/81b92cef1e23
eslintignore no longer needs to ignore directories with no js files. r=mossop
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.