credo-ts
credo-ts copied to clipboard
Switch from prettier + eslint to Biome
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:
- migration of all ESLint rules to Biome. Some may not be needed, but e.g. no restricted globals would help: https://github.com/openwallet-foundation/credo-ts/blob/main/.eslintrc.js#L78
Wait for all big PRs that are open to be merged?
- [ ] https://github.com/openwallet-foundation/credo-ts/pull/2127
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.
This is done 🙌 (thanks again @sairanjit )