chore: switch to biome for lint and format #2430
Merged
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.
Description
Switch to BiomeJS from ESLint & Prettier. Biome is simpler to configure, way faster and also smarter. It can catch bugs and find performance issues.
Tried to keep changes to lint rules in this PR to a minimum but included are:
Error
React.Fragment
sgetRef={(c): void => (qrRef.current = c)}
)const
where possibleconst ... of
toforEach
Checks are run pre-commit as before, can use scripts
lint:fix
&format:fix
to auto-fix issues where possible. Some of the rules inbiome.json
currently turned off would be useful to have but fixes need to be reviewed more closely.Type of change
Tests