Skip to content

chore: update rule count in readme#9374

Merged
dyc3 merged 1 commit intomainfrom
dyc3/update-rule-count
Mar 6, 2026
Merged

chore: update rule count in readme#9374
dyc3 merged 1 commit intomainfrom
dyc3/update-rule-count

Conversation

@dyc3
Copy link
Copy Markdown
Contributor

@dyc3 dyc3 commented Mar 6, 2026

Summary

Test Plan

Docs

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 6, 2026

⚠️ No Changeset found

Latest commit: bc63c40

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 6, 2026

Walkthrough

This pull request updates the README documentation across all supported language variants (English, Spanish, French, Japanese, Korean, Polish, Portuguese, Russian, Ukrainian, and Chinese). The primary change increases the documented linter rule count from 340+ to 450+. Several language versions also add descriptions clarifying that Biome can format and analyse malformed code in real-time within editors. All changes are documentation-only with no alterations to code or public declarations.

Possibly related PRs

Suggested labels

A-Linter

Suggested reviewers

  • siketyan
  • ematipico
🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description uses the standard PR template without custom content, but the template structure is related to documenting changes; however, no actual motivation or explanation is provided. Consider filling in the Summary section with the motivation for updating the rule count and why this change is necessary.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: updating the linter rule count from 340 to 450 across multiple README translations.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dyc3/update-rule-count

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/@biomejs/biome/README.fr.md (1)

46-47: ⚠️ Potential issue | 🟡 Minor

Note the terminology inconsistency with the English version.

The French text uses "analyser" (analyse) whereas the English README specifies "lint" for the same capability. Spanish and Portuguese-BR follow similar patterns. While both terms are technically accurate—linting is a form of code analysis—maintaining consistent terminology across language versions would improve clarity. Consider aligning with "linter" terminology or confirming if the broader "analyse" is intentional for this translation.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/`@biomejs/biome/README.fr.md around lines 46 - 47, Update the French
README text to use the same terminology as the English README by replacing
"analyser" (or "analyse") with the linter/lint terminology used in the English
version; specifically modify the sentence containing "Il peut formater et
analyser du code malformé pendant que vous l’écrivez." to use "lint" or "linter"
consistently (e.g., "Il peut formater et lint/faire du lint sur du code
malformé...") and ensure Spanish and Portuguese-BR files are updated similarly
if they currently use "analyser"/equivalent so all localized READMEs match the
English wording.
🧹 Nitpick comments (1)
packages/@biomejs/biome/README.fr.md (1)

43-47: Consider adding details to the PR description.

The PR description is empty beyond the template. It would be helpful to mention the rule count change and, if applicable, disclose any AI assistance used (as per project guidelines).

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/`@biomejs/biome/README.fr.md around lines 43 - 47, The PR
description is currently just the template and lacks important details; update
the PR description to state the change in the rule count (how many rules were
added/removed or the new total) and any other user-visible changes related to
the README.fr.md update, and if any AI assistance was used follow the project
disclosure policy by adding a short note; ensure the PR body references the
README.fr.md changes and the exact rule count delta so reviewers can verify the
documentation update.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@packages/`@biomejs/biome/README.fr.md:
- Around line 46-47: Update the French README text to use the same terminology
as the English README by replacing "analyser" (or "analyse") with the
linter/lint terminology used in the English version; specifically modify the
sentence containing "Il peut formater et analyser du code malformé pendant que
vous l’écrivez." to use "lint" or "linter" consistently (e.g., "Il peut formater
et lint/faire du lint sur du code malformé...") and ensure Spanish and
Portuguese-BR files are updated similarly if they currently use
"analyser"/equivalent so all localized READMEs match the English wording.

---

Nitpick comments:
In `@packages/`@biomejs/biome/README.fr.md:
- Around line 43-47: The PR description is currently just the template and lacks
important details; update the PR description to state the change in the rule
count (how many rules were added/removed or the new total) and any other
user-visible changes related to the README.fr.md update, and if any AI
assistance was used follow the project disclosure policy by adding a short note;
ensure the PR body references the README.fr.md changes and the exact rule count
delta so reviewers can verify the documentation update.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5e59323a-a792-4337-bcfe-a1ba68086c66

📥 Commits

Reviewing files that changed from the base of the PR and between 3c8996c and bc63c40.

📒 Files selected for processing (11)
  • packages/@biomejs/biome/README.es.md
  • packages/@biomejs/biome/README.fr.md
  • packages/@biomejs/biome/README.ja.md
  • packages/@biomejs/biome/README.kr.md
  • packages/@biomejs/biome/README.md
  • packages/@biomejs/biome/README.pl.md
  • packages/@biomejs/biome/README.pt-BR.md
  • packages/@biomejs/biome/README.ru.md
  • packages/@biomejs/biome/README.uk.md
  • packages/@biomejs/biome/README.zh-CN.md
  • packages/@biomejs/biome/README.zh-TW.md

Copy link
Copy Markdown
Member

@Netail Netail left a comment

Choose a reason for hiding this comment

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

Almost the 500 milestone 🤩

@dyc3 dyc3 merged commit dda9b3d into main Mar 6, 2026
6 checks passed
@dyc3 dyc3 deleted the dyc3/update-rule-count branch March 6, 2026 21:58
ematipico pushed a commit that referenced this pull request Mar 7, 2026
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