1701734 - Top-level dot files are not run through ESLint on continuous integration runs

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, defect, P2)

Description

[Mark Banner (:standard8)]

If I run ./mach eslint locally on latest central, then I am seeing:

.eslintrc.js
  134:23  error  Replace `⏎········"security/manager/ssl/tests/mochitest/browser/**",⏎······` with `"security/manager/ssl/tests/mochitest/browser/**"`  prettier/prettier (eslint)

However this does not show on treeherder. Back in bug 1608535 when we implemented parallelism for ESLint, we added the * on the command to make the parallelism work. At some stage since then we've fixed it so that it happens by default.

We can therefore remove the * and have the top-level dot files linted as well.

Example try run, which shows the ES build at 8 minutes, approximately the same as on mozilla-central:

https://treeherder.mozilla.org/jobs?repo=try&revision=7d335a3ba0baff3c2a22632f6bc1ba24d5abc765&selectedTaskRun=V0KukGnRRA2v0ggR4QSmGQ.0

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1701734 - Re-enable ESLint on top-level dot files. r?ahal!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b36d7f2046f2
Re-enable ESLint on top-level dot files. r=ahal

Comment 3

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/b36d7f2046f2