Closed Bug 1303525 Opened 4 years ago Closed 4 years ago
Ignore all of browser/branding/ for eslint (like we ignore all of browser/app)
58 bytes, text/x-review-board-request
The only JS-like files there are the pref brandings, and those aren't really JS (pref files are 'special') and might have preprocessing instructions (I added some in bug 1289172 and initially got backed out for them...). We should just be ignoring those files.
I agree we should just disable the pref files, I'm not convinced its a good idea to blanket those directories, as we could add something that does need it. Although people should check to see if a rule needs enabling, I could see it easily being forgotten, so I'd fall on the side of caution. I'm going to get a patch up for this, since it'll help the no-undef work.
Assignee: nobody → standard8
Attachment #8804322 - Flags: review?(jaws) → review?(gijskruitbosch+bugs)
Comment on attachment 8804322 [details] Bug 1303525 - Ignore pref js files for eslint in browser/branding. https://reviewboard.mozilla.org/r/88352/#review87602
Attachment #8804322 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/82b5112c0bc3 Ignore pref js files for eslint in browser/branding. r=Gijs
You need to log in before you can comment on or make changes to this bug.