Closed Bug 1271734 Opened 5 years ago Closed 5 years ago

Move all |mach eslint| infrastructure to tools/lint


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

Not set


(firefox50 fixed)

Tracking Status
firefox50 --- fixed


(Reporter: ahal, Assigned: ahal)




(1 file)

I'm working on a linting framework that will eventually provide a common interface to all our linting. This will include eslint (though don't worry, my aim will be 100% backwards compatibility with the existing setup).

Anyway, as part of that I'd like to put all lint related tools in tools/lint. The python/ is really meant for python related commands (like running an interactive python shell, or running python tests). Eslint doesn't belong in there. I'd also like to move the mozilla eslint plugin there too as it seems like a bit of a better fit than testing.
These two bugs are close to landing and will bitrot with this. So I'll wait for them to land before proceeding.
Depends on: 1265082, 1270851
This commit simply moves 'testing/eslint' to 'tools/lint/eslint' and the eslint related
mach command from 'python/' to 'tools/lint/'. It shouldn't
have any functional change on running eslint, either through mach or taskcluster.

This is in preparation for bug 1258341, to make the diffs there a little easier to read.

Review commit:
See other reviews:
Attachment #8759323 - Flags: review?(mratcliffe)
Hey Mike, sorry I pushed to review with the wrong bug number at first. I'm flagging you because I noticed you recently did some major eslint refactors. Let me know if I should flag someone else instead.

Here is a working try run with these changes:
Component: General → ESLint
Comment on attachment 8759323 [details]
Bug 1271734 - Move all eslint related infrastructure to tools/lint,
Attachment #8759323 - Flags: review?(mratcliffe) → review+
Pushed by
Move all eslint related infrastructure to tools/lint, r=miker
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
It would be nice to broadcast this change to platform and Firefox mailing lists, as some editors require absolute paths to the our ESLint setup, which will now need adjusting.
Sure, I'll send a note out.
Product: Testing → Firefox Build System
You need to log in before you can comment on or make changes to this bug.