[meta] Automatic ESLinting on MozReview

RESOLVED WORKSFORME

Status

defect
RESOLVED WORKSFORME
4 years ago
Last year

People

(Reporter: mconley, Unassigned)

Tracking

({meta})

Details

We should have ESLint run on all JS files that get pushed to MozReview.
Reporter

Updated

4 years ago
Depends on: 1229113, 1196263
Reporter

Updated

4 years ago
Depends on: 1229106
Reporter

Updated

4 years ago
Depends on: 1239045

Updated

4 years ago
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: Last year
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.