Closed Bug 1375166 Opened 6 years ago Closed 6 years ago

[mozlint] Can't run single file with |mach eslint|

Categories

(Developer Infrastructure :: Lint and Formatting, defect)

Product:
Developer Infrastructure
Component:
Lint and Formatting
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox56 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox56 --- fixed

People

(Reporter: ahal, Assigned: ahal)

References

Details

Attachments

(1 file)

Bug 1375166 - [mozlint] Don't require leading '.' in extensions,
59 bytes, text/x-review-board-request
standard8
: review+
Details 
This is a regression from bug 1288432. Running ./mach eslint <path/to/file.js> does not lint the file. This happens because mozlint is expecting the 'extensions' [1] to start with a period.

This is silly, mozlint should strip any leading periods if found.

[1] https://dxr.mozilla.org/mozilla-central/rev/e1e4a481b7e88dce163b9cccc2fb72032023befa/tools/lint/eslint.yml#8
Comment on attachment 8880059 [details]
Bug 1375166 - [mozlint] Don't require leading '.' in extensions,

https://reviewboard.mozilla.org/r/151388/#review156780

Looks great, thank you.
Attachment #8880059 - Flags: review?(standard8) → review+
Pushed by ahalberstadt@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2b190948b81b
[mozlint] Don't require leading '.' in extensions, r=standard8
https://hg.mozilla.org/mozilla-central/rev/2b190948b81b
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Product: Testing → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.