-
Notifications
You must be signed in to change notification settings - Fork 5
Biome install #66
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
Biome install #66
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Good work Nathan, thank you.
My only comments on this is if you could add a biome.json file? This will ensure all formatting is centralized (and not varying based on IDE-specific Biome extension settings).
I believe the command pnpm biome init
should do it. Thank you!
Gotcha-thought I had a biome.json but I'll add if it's missing when i get a moment |
So @coco-wizard -- running biome in the previous config defaults to the default biome.json, but doesn't save it if you don't want it. I added the biome.json so we can modify parameters as needed. pushing commit now! |
Biome is installed on the project! Changes in this PR:
.gitignore
to not track mac filesystem stuffNote that the CI workflow will not reformat code if it detects errors -- this possibility was tossed around but after hours of wrestling GitHub actions with no result, I gave in and setup husky instead.