8000 Biome install by nathansamuell · Pull Request #66 · codecoogs/web · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 10 commits into from
Mar 16, 2025
Merged

Biome install #66

merged 10 commits into from
Mar 16, 2025

Conversation

nathansamuell
Copy link
Contributor

Biome is installed on the project! Changes in this PR:

  • Biome CI workflow implemented, commits/prs will fail checks if they don't pass biome formatting
  • Husky Git Hook to automate formatting for devs (Husky is now a dev dependency)
  • Updated .gitignore to not track mac filesystem stuff
  • Formatted the entire project using biome to make sure existing code meets formatting specifications

Note 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.

Copy link
vercel bot commented Mar 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 9, 2025 11:46pm

@nathansamuell nathansamuell requested review from coco-wizard and removed request for coco-wizard March 4, 2025 20:11
@nathansamuell nathansamuell added enhancement New feature or request approved All reviews complete and changes approved labels Mar 4, 2025
Copy link
Contributor
@coco-wizard coco-wizard left a 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!

@nathansamuell
Copy link
8000 Contributor Author

Gotcha-thought I had a biome.json but I'll add if it's missing when i get a moment

@nathansamuell
Copy link
Contributor Author

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!

@coco-wizard coco-wizard merged commit 51ce577 into main Mar 16, 2025
4 checks passed
@coco-wizard coco-wizard deleted the biome-install branch March 16, 2025 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved All reviews complete and changes approved enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0