-
Notifications
You must be signed in to change notification settings - Fork 207
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
Switch from prettier + eslint to Biome #2145
Comments
Everything that helps to improve CI performance will be well received, and certainly eslint is getting very, very slow since some time. I don't have experience with Biome but I think we could give a try. |
Basically all projects I start now use Biome and i strongly prefer it over prettier+eslint. It does not require 10 plugins for everything to work, it is insanely fast and way easier to configure. I think we can introduce it with just the config some rules off. And slowly in following PRs turn each rule on with the fixes. Might be a nicer review process because of that. |
@sairanjit the big didcomm PR got merged, so feel free to take a stab at the biome setup! |
Sure @TimoGlastra I will pick this soon. |
Example of setup can be found here: https://github.com/openwallet-foundation-labs/oid4vc-ts/blob/main/biome.json
https://github.com/openwallet-foundation-labs/oid4vc-ts/blob/main/package.json#L14-L15
Things to consider:
Wait for all big PRs that are open to be merged?
The text was updated successfully, but these errors were encountered: