+
Skip to content

docs: fix broken link, update rule count #7194

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 12, 2025
Merged

docs: fix broken link, update rule count #7194

merged 1 commit into from
Aug 12, 2025

Conversation

9romise
Copy link
Contributor

@9romise 9romise commented Aug 12, 2025

Summary

Test Plan

Docs

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Aug 12, 2025

⚠️ No Changeset found

Latest commit: b6b1142

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 coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 12, 2025 •
edited
Loading

Walkthrough

Multiple locale README files for Biome were updated to adjust the linter rules count (to ~340/340+) and to change the rules documentation URL to the language-specific /linter/javascript/rules/ path. Some locales (e.g., PT-BR, RU) also added or tweaked sentences about diagnostics and live formatting; PT-BR removed a prior editor-interactivity line. zh-TW wrapped the rule count in a hyperlink. No code or public API changes.

Suggested labels

A-Linter, L-JavaScript

Suggested reviewers

  • siketyan
  • dyc3
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

coderabbitai[bot]
coderabbitai bot reviewed Aug 12, 2025
View reviewed changes
Copy link
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.

Actionable comments posted: 0

🔭 Outside diff range comments (1)
packages/@biomejs/biome/README.pt-BR.md (1)

41-46: Avoid triple “Biome é …” repetition; minor copy tweaks

Both links return 200 OK. Nice fixes—let’s improve flow and tweak one noun:

-**Biome é um [linter eficiente](https://github.com/biomejs/biome/tree/main/benchmark#linting)** para _JavaScript_, _TypeScript_, e _JSX_ que possui **[mais de 340 regras](https://biomejs.dev/pt-br/linter/javascript/rules/)** do ESLint, typescript-eslint, e de [outras fontes](https://github.com/biomejs/biome/discussions/3).
+**O Biome também é um [linter eficiente](https://github.com/biomejs/biome/tree/main/benchmark#linting)** para _JavaScript_, _TypeScript_ e _JSX_ que possui **[mais de 340 regras](https://biomejs.dev/pt-br/linter/javascript/rules/)** do ESLint, typescript-eslint e de [outras fontes](https://github.com/biomejs/biome/discussions/3).

-Ele **fornece diagnósticos detalhados e contextualizados** que ajudam você a melhorar seu código e se tornar um programador melhor!
+Ele **fornece diagnósticos detalhados e contextualizados** que ajudam você a melhorar seu código e se tornar um programador melhor!

-Isso permite formatar e lintar códigos malformados enquanto você programa.
+Isso permite formatar e lintar código malformado enquanto você programa.
🧹 Nitpick comments (2)
packages/@biomejs/biome/README.zh-TW.md (1)

41-41: Confirm locale path availability for rules URL

Link currently targets the non-localised page. If a zh-TW rules page exists, consider switching to /zh-tw/linter/javascript/rules/ for consistency with other locales. Otherwise, this is fine.

If you want, I can check all updated links resolve with a quick script.

packages/@biomejs/biome/README.ru.md (1)

41-46: Russian copy: tighten wording (“более лучшим” → “лучшим”)

Minor style fix; “более лучшим” is redundant in Russian.

Apply this diff:

-Он **выводит подробную диагностику с контекстной информацией**, которая помогает вам улучшить ваш код и стать более лучшим программистом!
+Он **выводит подробную диагностику с контекстной информацией**, которая помогает вам улучшить ваш код и стать лучшим программистом!
📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1a4279a and b6b1142.

📒 Files selected for processing (10)
  • packages/@biomejs/biome/README.fr.md (1 hunks)
  • packages/@biomejs/biome/README.hi.md (1 hunks)
  • packages/@biomejs/biome/README.ja.md (1 hunks)
  • packages/@biomejs/biome/README.kr.md (1 hunks)
  • packages/@biomejs/biome/README.md (1 hunks)
  • packages/@biomejs/biome/README.pt-BR.md (1 hunks)
  • packages/@biomejs/biome/README.ru.md (1 hunks)
  • packages/@biomejs/biome/README.uk.md (1 hunks)
  • packages/@biomejs/biome/README.zh-CN.md (1 hunks)
  • packages/@biomejs/biome/README.zh-TW.md (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-08-05T14:43:29.581Z 
Learnt from: dyc3
PR: biomejs/biome#7081
File: packages/@biomejs/biome/configuration_schema.json:7765-7781
Timestamp: 2025-08-05T14:43:29.581Z
Learning: The file `packages/biomejs/biome/configuration_schema.json` is auto-generated and should not be manually edited or reviewed for schema issues; any changes should be made at the code generation source.

Applied to files:

  • packages/@biomejs/biome/README.md
  • packages/@biomejs/biome/README.hi.md
🪛 LanguageTool
packages/@biomejs/biome/README.fr.md

[style] ~43-~43: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...le.algora.io/challenges/prettier)**. **Biome est un [outil de linting performant](ht...

(ENGLISH_WORD_REPEAT_BEGINNING_RULE)

packages/@biomejs/biome/README.kr.md

[style] ~41-~41: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...ra.io/challenges/prettier)을** 자랑합니다. Biome은 JavaScript, TypeScript, JSX, _...

(ENGLISH_WORD_REPEAT_BEGINNING_RULE)

packages/@biomejs/biome/README.pt-BR.md

[style] ~41-~41: Três frases seguidas começam com a mesma palavra. Considere reformular a frase ou use um dicionário para encontrar um sinônimo.
Context: ...le.algora.io/challenges/prettier)**. **Biome é um [linter eficiente](https://github....

(PORTUGUESE_WORD_REPEAT_BEGINNING_RULE)

packages/@biomejs/biome/README.md

[style] ~41-~41: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...le.algora.io/challenges/prettier)**. **Biome is a [performant linter](https://github...

(ENGLISH_WORD_REPEAT_BEGINNING_RULE)

🔇 Additional comments (8)
packages/@biomejs/biome/README.zh-CN.md (1)

41-41: LGTM: rule count and link fixed correctly

Count updated to “340 余项” and link points to the locale-specific javascript rules page. Matches PR intent.

packages/@biomejs/biome/README.ja.md (1)

41-41: LGTM: updated to 340+ rules with correct locale URL

The new link targets the JA rules index and the phrasing “340以上のルール” reads well.

packages/@biomejs/biome/README.hi.md (1)

41-41: LGTM: rule count and rules link updated

Accurately reflects “३४० से अधिक नियम” and points to the consolidated rules index.

packages/@biomejs/biome/README.uk.md (1)

41-42: LGTM: updated count and locale-specific URL

“понад 340 правил” with /uk/linter/javascript/rules/ is spot on. Thanks for keeping it consistent.

packages/@biomejs/biome/README.fr.md (1)

43-44: LGTM: FR rules URL and 340+ count

Correctly switched to /fr/linter/javascript/rules/ and updated the count. All good on my side.

packages/@biomejs/biome/README.md (2)

41-42: LGTM: broken link fixed and count bumped

Moved to /linter/javascript/rules/ and “more than 340 rules” reads clean. Nicely done.

41-41: All locale rule links return HTTP 200 ✅

All tested URLs—including default and localized paths (ja, fr, uk, zh-cn)—respond with 200. No broken links detected, so no further changes needed.

packages/@biomejs/biome/README.kr.md (1)

41-41: LGTM: KR paragraph updated correctly

“340개 이상의 규칙” with the consolidated rules URL looks correct and matches the PR’s goal.

@siketyan siketyan merged commit fcea68a into biomejs:main Aug 12, 2025
3 checks passed
@9romise 9romise deleted the readme branch August 12, 2025 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@siketyan siketyan siketyan approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@9romise @siketyan
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载