fix(flat): remove unused deprecation plugin import - #229
Merged
Conversation
eslint-plugin-deprecation v3 bundles an older @typescript-eslint/utils that references LegacyESLint, which was removed in ESLint 10. The import was unused — consuming projects are expected to configure their own deprecation rule with the correct tsconfig path. Use @typescript-eslint/no-deprecated in consuming projects instead.
…nal peer dep - Add inline comment in flat.mjs explaining why eslint-plugin-deprecation is excluded and pointing to @typescript-eslint/no-deprecated as the replacement - Mark eslint-plugin-deprecation as optional in peerDependenciesMeta so flat config consumers don't get spurious peer dep warnings
xiyu95
enabled auto-merge (squash)
April 7, 2026 15:13
darrenjaneczek
approved these changes
Apr 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
eslint-plugin-deprecationimport fromflat.mjseslint-plugin-deprecation@3bundles an older@typescript-eslint/utilsthat referencesLegacyESLint, which was removed in ESLint 10 — importing the plugin crashes ESLint 10 even though it was never used in any config object@typescript-eslint/no-deprecated(the modern equivalent, available in@typescript-eslintv7+) directly in their owneslint.config.mjsTest plan
flat.mjsworks without errors under ESLint 10@typescript-eslint/no-deprecatedas the replacement rule