Skip to content

Remove vestigial ESLint config dependencies - #141

Merged
tbroadley merged 1 commit into
masterfrom
fix-eslint-peer-deps
Sep 6, 2026
Merged

tbroadley merged 1 commit into
masterfrom
fix-eslint-peer-deps

Conversation

@tbroadley

Copy link
Copy Markdown
Owner

Problem

yarn install printed two peer-dependency warnings:

warning " > eslint-config-airbnb-base@15.0.0" has incorrect peer dependency "eslint@^7.32.0 || ^8.2.0".
warning " > eslint-plugin-import@2.32.0" has incorrect peer dependency "eslint@^2 || ... || ^9".

What was vestigial vs. used

Both packages are vestigial. They were only ever referenced by .eslintrc.yml (extends: airbnb-base, plugin:import/typescript, and the import/* rules). The repo has since moved to a flat config, and eslint.config.js does not reference either package — it composes @eslint/js recommended, @typescript-eslint recommended, and a handful of custom rules. ESLint 10 no longer reads eslintrc files at all, so .eslintrc.yml was dead config and is removed here too.

Why not upgrade instead

No published release of either package supports ESLint 10:

  • eslint-config-airbnb-base — latest is 15.0.0, peer range eslint@^7.32.0 || ^8.2.0
  • eslint-plugin-import — latest is 2.32.0, peer range tops out at eslint@^9

Since the flat config never loaded them, removing them costs zero rule coverage.

Changes

  • Remove eslint-config-airbnb-base and eslint-plugin-import from devDependencies
  • Delete the stale .eslintrc.yml
  • Drop the now-dangling .eslintrc.yml reference from a comment in eslint.config.js
  • Add a ## [Unreleased] CHANGELOG entry

Verification

  • yarn install — clean, no peer-dependency warnings
  • yarn ci — spellcheck clean, lint clean, 101 mocha tests passing
  • Sanity-checked that lint still reports real problems: a scratch .ts file with unused vars and an undefined identifier was flagged by @typescript-eslint/no-unused-vars and no-undef, then removed. The config is not silently linting nothing.

🤖 Generated with Claude Code

@tbroadley
tbroadley force-pushed the fix-eslint-peer-deps branch from e514b4e to fab9b90 Compare September 6, 2026 20:55
`eslint-config-airbnb-base` and `eslint-plugin-import` were left over from
the old `.eslintrc.yml` setup. The active flat config (`eslint.config.js`)
does not reference either package, and ESLint 10 no longer reads eslintrc
files at all, so the config file and both dependencies were dead weight.

Neither package has a release whose peer range includes ESLint 10
(eslint-config-airbnb-base@15.0.0 allows ^7.32.0 || ^8.2.0;
eslint-plugin-import@2.32.0 allows up to ^9), so removing them is the
only way to clear the warnings, and it costs no rule coverage since the
flat config never loaded them.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@tbroadley
tbroadley force-pushed the fix-eslint-peer-deps branch from fab9b90 to e015aca Compare September 6, 2026 20:58
@tbroadley
tbroadley merged commit 26b573f into master Sep 6, 2026
6 checks passed
@tbroadley
tbroadley deleted the fix-eslint-peer-deps branch September 6, 2026 20:59
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