-
-
Notifications
You must be signed in to change notification settings - Fork 760
fix(RELEASES.md): grammar and typo error #7993
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
Conversation
|
WalkthroughThis PR updates RELEASES.md with several grammatical and wording corrections: changes “enforcing deadlines to a group of volunteers” → “enforcing deadlines for a group of volunteers”; “they are meant for testing and verify that bugs are fixed” → “they are meant for testing and verifying that bugs are fixed”; fixes “rights” → “right” in Beta and Stable release sections; and simplifies “one single version” → “a single version”. No code or API surface changes. Suggested reviewers
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (1 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🧰 Additional context used🧠 Learnings (2)📚 Learning: 2025-10-15T09:24:31.042ZApplied to files:
📚 Learning: 2025-10-15T09:22:46.002ZApplied to files:
🪛 LanguageToolRELEASES.md[style] ~5-~5: To make your writing clearer, consider a shorter, more direct phrase. (ADVERSE_EFFECT_ON) [uncategorized] ~11-~11: Do not mix variants of the same word (‘pre-release’ and ‘prerelease’) within a single text. (EN_WORD_COHERENCY) 🔇 Additional comments (5)
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (1)
RELEASES.md (1)
11-11: Terminology inconsistency: Consider standardising hyphenation.The section heading uses "Prerelease" (no hyphen) whilst line 11 uses "pre-releases" (hyphenated). Choose one variant consistently throughout the document.
Static analysis also flags this. If you prefer the non-hyphenated form:
-We publish pre-releases of the main `@biomejs/biome` package twice a week. +We publish prerelease versions of the main `@biomejs/biome` package twice a week.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
RELEASES.md(3 hunks)
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-10-15T09:24:31.042Z
Learnt from: CR
Repo: biomejs/biome PR: 0
File: crates/biome_parser/CONTRIBUTING.md:0-0
Timestamp: 2025-10-15T09:24:31.042Z
Learning: Applies to crates/biome_parser/crates/biome_*_{syntax,factory}/** : Create per-language crates biome_<lang>_syntax and biome_<lang>_factory under crates/
Applied to files:
RELEASES.md
📚 Learning: 2025-10-15T09:22:46.002Z
Learnt from: CR
Repo: biomejs/biome PR: 0
File: crates/biome_js_formatter/CONTRIBUTING.md:0-0
Timestamp: 2025-10-15T09:22:46.002Z
Learning: Applies to crates/biome_js_formatter/**/Cargo.toml : Declare the dependency `biome_js_formatter = { version = "0.0.1", path = "../biome_js_formatter" }` for internal installation
Applied to files:
RELEASES.md
🪛 LanguageTool
RELEASES.md
[style] ~5-~5: To make your writing clearer, consider a shorter, more direct phrase.
Context: ...olunteers is counterproductive, and can have negative effects on people. ## npm packages ### Prerele...
(ADVERSE_EFFECT_ON)
[uncategorized] ~11-~11: Do not mix variants of the same word (‘pre-release’ and ‘prerelease’) within a single text.
Context: ...mpackages ### Prerelease We publish pre-releases of the main@biomejs/biome` package tw...
(EN_WORD_COHERENCY)
🔇 Additional comments (1)
RELEASES.md (1)
5-5: Grammatical corrections look good. The preposition fix (deadlines for vs to), parallel verb form (verifying), and redundancy removal (a single vs one single) all improve clarity and consistency.Also applies to: 11-11, 40-40
Summary
Test Plan
Docs