Skip to content

Conversation

camdecoster
Copy link
Contributor

@camdecoster camdecoster commented Aug 19, 2025

Description

Update Biome dependency to v2.2.0 and migrate the config file to be compatible with the new version.

Changes

  • Update Biome to v2.2.0 from v1.8.3
  • Update Biome config
  • Remove duplicated keys in JSON test mock

Notes

  • Users will need to run npm i after this merges to get the new Biome binary
  • I'm going to start formatting on save. This will result in a number of changes to follow the rules in the Biome config. The philosophy of Biome formatting is to have a limited number of options so that you set it and forget it and don't argue about what set up is better. Let's use that same philosophy!
  • FYI, if(someBoolean) will get switched to if (someBoolean) because there's no way to configure that space in between if and (
  • It would be nice to add a linting/formatting check in CI in the future
  • The current rules are being mostly ignored because we don't enforce any check This isn't true. There is a linting check, but there's no formatting check.

@camdecoster camdecoster added the dependencies Pull requests that update a dependency file label Aug 20, 2025
@gvwilson gvwilson requested a review from emilykl August 22, 2025 15:16
@gvwilson gvwilson added P1 needed for current cycle fix fixes something broken labels Aug 22, 2025
Copy link
Contributor

@emilykl emilykl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @camdecoster 🚀

Check that one mock file, otherwise looks good

@camdecoster camdecoster merged commit 86eb77b into master Aug 25, 2025
6 of 7 checks passed
@camdecoster camdecoster deleted the cam/update-biome branch August 25, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file fix fixes something broken P1 needed for current cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants