1917532 - Remove overrides from test configurations in eslint-plugin-mozilla

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task, P3)

Description

[Mark Banner (:standard8)]

In the flat configuration, plugins cannot specify the equivalent of overrides. We already strip the overrides in the top-level .eslintrc.js and then manually specify the equivalent, so I think we can make this simpler on ourselves and have eslint-plugin-mozilla specify only the main configuration.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1917532 - Remove overrides sections from *-test configurations in eslint-plugin-mozilla. r?#frontend-codestyle-reviewers!

ESLint v9 doesn't support overrides/file specification in the configurations, so we don't want these
now.

The top-level configuration already strips out the overrides for these and handles them separately, so
there's nothing to add - simply remove the 'removeOverrides' function.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8d6f7bc74d08
Remove overrides sections from *-test configurations in eslint-plugin-mozilla. r=frontend-codestyle-reviewers,mossop

Comment 3

[Sandor Molnar[:smolnar]]

Backed out for causing node newtab failures

Backout link: https://hg.mozilla.org/integration/autoland/rev/26f637a20dc180da705cb962f1c6acc84cd98277

Push with failures

Failure log -> TEST-UNEXPECTED-FAIL | bundles | about:asrouter bundle out of date

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/70aa76e5ff44
Remove overrides sections from *-test configurations in eslint-plugin-mozilla. r=frontend-codestyle-reviewers,mossop

Comment 5

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/70aa76e5ff44