1239044 - [meta] Automatic ESLinting on MozReview

Status: RESOLVED WORKSFORME

(MozReview Graveyard :: General, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

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

Comment 1

[Andrew Halberstadt [:ahal] (PTO Until Apr 3rd)]

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.

Comment 2

[Mark Banner (:standard8)]

This has been enabled for a while now, fixed in a different bug/issue somewhere.