Repository navigation
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 31 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (2)
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 |
5e3dc11 to
d3219c1
Compare
d3219c1 to
b175d47
Compare
b175d47 to
c6a585a
Compare
c6a585a to
8ce2558
Compare
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The tests correctly exercise multi-rule linting and autofix behavior without unresolved issues.
Review effort: Balanced
Findings: None
What changed in this PR
Adds end-to-end coverage for interactions among shipped Bash rules and the recommended ESLint configuration.
Changes:
- Tests recommended-config reporting and clean scripts.
- Verifies multiple rules and autofixes compose correctly.
| File | Description |
|---|---|
tests/plugin.test.ts |
Adds integration coverage for shipped rules and recommended configuration. |
tests/autofix.test.ts |
Tests composing fixes from multiple rules. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
8ce2558 to
6a57b30
Compare
9da9753 to
803444c
Compare
803444c to
ec8518c
Compare
ec8518c to
3d0c6ad
Compare
3d0c6ad to
bcc9bf2
Compare
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
bcc9bf2 to
45e55b1
Compare
Adds end-to-end tests that run several shipped rules together through ESLint's
Linter. They live at the top of the stack because they need every rule to exist. No source files change in this PR.What's included
tests/plugin.test.tsgains a "shipped rules" block that checks:no-unused-vars,require-cd-guard,require-read-r, andno-unquoted-expansions;tests/autofix.test.tsgains a multi-ruleverifyAndFixtest showing that fixes from different rules compose. The first pass turns`pwd`into$(pwd), and the second pass quotes it as"$(pwd)".Testing
5 new tests (368 total); build, lint, and format checks pass.
🤖 Generated with Claude Code
Stack created with GitHub Stacks CLI • Give Feedback 💬