Closed Bug 1279210 Opened 4 years ago Closed 4 years ago

mach eslint --setup can no longer install eslint-plugin-mozilla after move to tools/lint

Categories

(Firefox Build System :: Lint and Formatting, defect)

3 Branch
defect
Not set

Tracking

(firefox50 fixed)

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

Details

Attachments

(1 file)

Follow up to 1271734.
./mach eslint --setup because it tries to locate eslint-plugin-mozilla in testing/esling.

A path needs updating in tools/lint/mach_commands.py
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Comment on attachment 8761557 [details]
Bug 1279210 - eslint setup: fix eslint module path;

https://reviewboard.mozilla.org/r/58664/#review55660

Ah, sorry about that! I did a dxr for 'testing/eslint' but evidently neglected join.
Attachment #8761557 - Flags: review?(ahalberstadt) → review+
Thanks for finding/fixing. I took the liberty of autolanding this.
(In reply to Andrew Halberstadt [:ahal] from comment #3)
> Thanks for finding/fixing. I took the liberty of autolanding this.

Did this not work? I don't see any commits here?
Flags: needinfo?(jdescottes)
Flags: needinfo?(ahalberstadt)
Somehow not recorded here by pulsebot, but:

https://hg.mozilla.org/integration/mozilla-inbound/rev/7fc8f70a567d4492607906d40532ef4a7124c8f5
Flags: needinfo?(jdescottes)
Flags: needinfo?(ahalberstadt)
https://hg.mozilla.org/mozilla-central/rev/7fc8f70a567d
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Product: Testing → Firefox Build System
Version: Version 3 → 3 Branch
You need to log in before you can comment on or make changes to this bug.