Skip to content

chore: migrate ESLint to flat config - #103

Open
liamhughes wants to merge 1 commit into
mainfrom
liamhughes/convert-eslint-config
Open

chore: migrate ESLint to flat config#103
liamhughes wants to merge 1 commit into
mainfrom
liamhughes/convert-eslint-config

Conversation

@liamhughes

Copy link
Copy Markdown
Contributor

Background

Open Renovate PR #102 can not be merged because ESLint removes support for .eslintrc.js for configuration.

Changes

Translates the existing config as-is: the three extends become a composed config array, and .eslintignore plus ignorePatterns collapse into ignores.

ESLint 9 made eslint.config.js the default and v10 removes eslintrc
support entirely, including the ESLINT_USE_FLAT_CONFIG=false escape
hatch, so .eslintrc.js has to go before eslint can be bumped past v8.

Translates the existing config as-is: the three extends become a
composed config array, and .eslintignore plus ignorePatterns collapse
into ignores. The same 12 files are linted with the same rules and no
new findings.

The eslintrc env node/es6 setting is dropped rather than translated to
languageOptions.globals. Only no-undef and no-redeclare consume the
globals list, and typescript-eslint's eslint-recommended overrides turn
both off for .ts files, so configuring globals has no effect on lint
results here. It becomes necessary only if linting is widened to the
root .js files.

Swaps @typescript-eslint/eslint-plugin for the typescript-eslint meta
package, which bundles the parser and exposes the flat configs, and
adds @eslint/js for eslint:recommended.
Comment thread .eslintignore
@@ -1,4 +0,0 @@
node_modules/*

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now ignored by default.

Comment thread .eslintignore
@@ -1,4 +0,0 @@
node_modules/*
**/package.json

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are only linting *.ts files.

@liamhughes
liamhughes marked this pull request as ready for review July 31, 2026 01:31
@liamhughes
liamhughes requested a review from a team as a code owner July 31, 2026 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant