Skip to content

Conversation

nginx-jack
Copy link
Collaborator

Biome: Use biome.json version as main

This PR does not update the biome version

Adds install-help target, which given a local version of 2.2.2, and a biome.json version of 2.2.3, will output the following:

❯ make install-help

🛠️  Install options for biome v2.2.3:

1) 🧪 Install manually via Homebrew (advanced):
   # Find the correct commit SHA for biome 2.2.3 here:
   # https://github.com/Homebrew/homebrew-core/commits/master/Formula/biome.rb
   curl -sSL https://raw.githubusercontent.com/Homebrew/homebrew-core/<COMMIT_SHA>/Formula/biome.rb -o biome.rb
   brew install ./biome.rb
   brew pin biome

2) 🚀 Install via npm (global):
   npm install -g @biomejs/[email protected]

3) 🧰 Use temporarily via npx (auto fallback in Makefile):
   npx -y @biomejs/[email protected] <command>

Note: This Makefile will automatically fall back to npx if the right version is not found locally.

@nginx-jack nginx-jack requested a review from a team as a code owner September 10, 2025 15:07
Copy link

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/nginx-hugo-theme/395/

Copy link
Collaborator

@lamATnginx lamATnginx left a comment

Choose a reason for hiding this comment

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

Add biome.rb to .gitignore

Copy link
Member

@TasoOneAsia TasoOneAsia left a comment

Choose a reason for hiding this comment

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

Non blocking, but we should document these initial setup commands somewhere (maybe in a "Local development" section in the README? open to suggestions).

@nginx-jack nginx-jack merged commit 822077a into main Sep 12, 2025
8 checks passed
@nginx-jack nginx-jack deleted the biome-at-it-again branch September 12, 2025 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants