Conversation
🦋 Changeset detectedLatest commit: b7a63c4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
WalkthroughThis PR adds a changeset file and updates the Possibly related PRs
Suggested labels
Suggested reviewers
🚥 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)
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 @.changeset/large-buses-stick.md:
- Line 5: The sentence "Added eslint-plugin-unicorn's `no-nested-ternary` as a
rule source for
[`noNestedTernary`](https://biomejs.dev/linter/rules/no-nested-ternary/)" in the
changeset description is missing a trailing period; edit that line in
.changeset/large-buses-stick.md to append a full stop at the end of the sentence
so it conforms to the changeset punctuation guideline.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: a1ec545d-734f-4b9b-b092-0c4320f89c5a
⛔ Files ignored due to path filters (1)
crates/biome_cli/src/execute/migrate/eslint_any_rule_to_biome.rsis excluded by!**/migrate/eslint_any_rule_to_biome.rsand included by**
📒 Files selected for processing (2)
.changeset/large-buses-stick.mdcrates/biome_js_analyze/src/lint/style/no_nested_ternary.rs
| "@biomejs/biome": patch | ||
| --- | ||
|
|
||
| Added eslint-plugin-unicorn's `no-nested-ternary` as a rule source for [`noNestedTernary`](https://biomejs.dev/linter/rules/no-nested-ternary/) |
There was a problem hiding this comment.
Missing full stop.
As per coding guidelines, every sentence in changeset descriptions should end with a full stop (period).
Proposed fix
-Added eslint-plugin-unicorn's `no-nested-ternary` as a rule source for [`noNestedTernary`](https://biomejs.dev/linter/rules/no-nested-ternary/)
+Added eslint-plugin-unicorn's `no-nested-ternary` as a rule source for [`noNestedTernary`](https://biomejs.dev/linter/rules/no-nested-ternary/).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.changeset/large-buses-stick.md at line 5, The sentence "Added
eslint-plugin-unicorn's `no-nested-ternary` as a rule source for
[`noNestedTernary`](https://biomejs.dev/linter/rules/no-nested-ternary/)" in the
changeset description is missing a trailing period; edit that line in
.changeset/large-buses-stick.md to append a full stop at the end of the sentence
so it conforms to the changeset punctuation guideline.
Merging this PR will not alter performance
Comparing Footnotes
|
Summary
Does what it says on the tin. eslint unicorn has a rule that does the same thing, but behaves slightly differently.
Test Plan
ci should be green
Docs