Fix eslint/espree parsing errors for ignored directories in browser/

RESOLVED FIXED in Firefox 52

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
Firefox 52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

Looking at the directories we ignore for eslint, there's a few issues that cause the espree parser to fail, and as a result, we don't see the normal eslint errors when un-ignoring those directories. Hence its difficult to assess the amount of work required.

Fixing these are fairly easy, I have a patch upcoming.
Comment hidden (mozreview-request)
Comment on attachment 8800607 [details]
Bug 1309842 - Fix eslint/espree parsing errors for ignored directories in browser/.

https://reviewboard.mozilla.org/r/85504/#review84184
Attachment #8800607 - Flags: review?(jaws) → review+

Comment 3

3 years ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dcfc73c043c0
Fix eslint/espree parsing errors for ignored directories in browser/. r=jaws

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/dcfc73c043c0
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
You need to log in before you can comment on or make changes to this bug.