Conversation
✅ Deploy Preview for biomejs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
|
Here we follow the same flow of the core repository. We send the PRs to the next branch, so we merge code a docs together |
b8cad4f to
49e7b7e
Compare
49e7b7e to
91d4050
Compare
WalkthroughA documentation callout tip has been added to the manual installation guide. The tip explains how users can upgrade Biome to the latest version using the 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/content/docs/guides/manual-installation.mdx`:
- Around line 87-89: The tip block currently shows the bare command "biome
upgrade" which is inconsistent with the manual-install flow that uses the local
binary invocation "./biome"; update the tip text inside the :::tip block so the
command reads "./biome upgrade" (and scan the manual-installation guide for any
other occurrences of the string "biome upgrade" and replace them with "./biome
upgrade" to keep invocation style consistent).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 96041bb2-07a7-4720-a3a5-8597647539d5
📒 Files selected for processing (1)
src/content/docs/guides/manual-installation.mdx
| :::tip | ||
| To **upgrade** Biome to the latest version, run `biome upgrade` | ||
| ::: |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify command-style consistency in manual installation docs.
rg -n -C2 'Now you can use Biome by simply running `\./biome`|`biome upgrade`|`\./biome upgrade`' src/content/docs/guides/manual-installation.mdxRepository: biomejs/website
Length of output: 217
🏁 Script executed:
# Check the surrounding context (lines 75-95) to see if PATH is mentioned
sed -n '75,95p' src/content/docs/guides/manual-installation.mdx | cat -nRepository: biomejs/website
Length of output: 736
🏁 Script executed:
# Search for PATH references in the entire file
rg -i 'path|environment|add.*to' src/content/docs/guides/manual-installation.mdx -nRepository: biomejs/website
Length of output: 41
🏁 Script executed:
# Check how `biome upgrade` is documented elsewhere
rg -l 'biome upgrade' src/content/docs/ | head -10Repository: biomejs/website
Length of output: 106
Use consistent invocation style for manual installations
Line 88 uses biome upgrade, but line 85 instructs users to run ./biome. For consistency with the manual-install flow (where the binary is in the current directory), change this to ./biome upgrade.
Suggested change
:::tip
-To **upgrade** Biome to the latest version, run `biome upgrade`
+To **upgrade** Biome to the latest version, run `./biome upgrade`
:::📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| :::tip | |
| To **upgrade** Biome to the latest version, run `biome upgrade` | |
| ::: | |
| :::tip | |
| To **upgrade** Biome to the latest version, run `./biome upgrade` | |
| ::: |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/content/docs/guides/manual-installation.mdx` around lines 87 - 89, The
tip block currently shows the bare command "biome upgrade" which is inconsistent
with the manual-install flow that uses the local binary invocation "./biome";
update the tip text inside the :::tip block so the command reads "./biome
upgrade" (and scan the manual-installation guide for any other occurrences of
the string "biome upgrade" and replace them with "./biome upgrade" to keep
invocation style consistent).
Summary
This PR adds instructions about upgrading Biome for users that install Biome manually.
Merge after biomejs/biome#9546