Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate eslint-config-next to eslint-plugin-import-x and eslint-react #123

Open
mcmxcdev opened this issue Oct 3, 2024 · 3 comments
Open

Comments

@mcmxcdev
Copy link

mcmxcdev commented Oct 3, 2024

I just saw that eslint-config-next pulls in eslint-plugin-import, eslint-plugin-jsx-a11y and eslint-plugin-react. These dependencies are known to be full of bloat dependencies, polyfills and support for ancient node versions.

See: https://github.com/vercel/next.js/blob/b68c188450f1d4ed932e9e28e9be2b5562bfe306/packages/eslint-config-next/package.json#L19-L21

Although probably not a 100% replacement in terms of rulesets, I would suggest moving from:

@mcmxcdev mcmxcdev changed the title Migrate eslint-config-next to eslint-plugin-import-x and eslint-react Migrate eslint-config-next to eslint-plugin-import-x and eslint-react Oct 3, 2024
@43081j
Copy link
Collaborator

43081j commented Oct 4, 2024

Good find 👍

Id love to see a jsx-a11y alternative too. As far as I know, nobody has built one yet

@mcmxcdev
Copy link
Author

mcmxcdev commented Oct 4, 2024

Biome does have accessibility rules ported over from eslint-plugin-jsx-a11y, see https://biomejs.dev/linter/rules/#accessibility but projects would need to be open to embrace another tool rather than just another eslint plugin.

@43081j
Copy link
Collaborator

43081j commented Oct 4, 2024

Yup I think we still need to provide a solution for eslint users

We can recommend both but they will be a preference for many years to come. So we need to make sure we satisfy both sides

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

No branches or pull requests

2 participants