-
Notifications
You must be signed in to change notification settings - Fork 375
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
[Feature]: Replace ESLint with Biome #5506
Comments
Hello @chenjiahan , I'm curious why not oxclint since it's already part of the ecosystem? |
@kirillbashtenko Both Biome and OXC can be used to speed up linting, Modern.js choose to use Biome because it can be used for both linting and formatting. Here is a discussion for comparison: biomejs/biome#1281 |
Thank you, now it's clear! |
@chenjiahan thank you for opening the issue. Would love to use biome with modernjs for our code bases |
bumpinng |
@chenjiahan Is it planned to have a dedicated set of "default" fields or will you use the standard biome "recommended" list as default? |
I prefer following the standard biome "recommended" list because it is well designed. |
Thank you for this 🎉 |
What problem does this feature solve?
ESLint has obvious performance issues in large monorepos like modern.js.
We can use Biome as the linter to replace ESLint, which can reduce dependencies and improve the lint performance.
Ref:
What does the proposed API look like?
@modern-js/eslint-config
package and reduce the maintenance overhead.The text was updated successfully, but these errors were encountered: