Closed Bug 1239044 Opened 9 years ago Closed 6 years ago

[meta] Automatic ESLinting on MozReview

Categories

(MozReview Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: mconley, Unassigned)

References

Details

(Keywords: meta)

We should have ESLint run on all JS files that get pushed to MozReview.
Depends on: 1239659
Product: Developer Services → MozReview
See also bug 1230962 and bug 1258341. The patches there are fully functional, but haven't landed yet.

The intent of those two bugs is to be able to run multiple linters at the same time. So for example, you could run both eslint and flake8 against all the files touched by a commit for example. The other intent is to have a single standardized way of running linters. The end goal is for someone to be able to check-in, e.g a Java linter and automatically get mach, mozreview and taskcluster integration for free.
This has been enabled for a while now, fixed in a different bug/issue somewhere.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.