chore(deps-dev): bump @vitest/eslint-plugin to 1.3.20 (supersedes #7701)#8305
chore(deps-dev): bump @vitest/eslint-plugin to 1.3.20 (supersedes #7701)#8305jherr wants to merge 1 commit into
Conversation
Applies the intent of #7701 directly on top of main instead of merging that stale branch, which would have downgraded vitest to 3.2.4. The ^8.46.1 @typescript-eslint dependency resolves as a nested subtree, so the top-level typescript-eslint stack stays at 8.45.0 and vitest at 3.2.4. No .npmrc needed. 1.3.20 enables the prefer-called-exactly-once-with rule; the flagged test assertions are collapsed into toHaveBeenCalledExactlyOnceWith via eslint --fix + prettier. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed due to a network error. 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 |
📊 Benchmark resultsComparing with 30ba6a1
|
Standalone replacement for #7701, applied directly on top of
main.Why not just merge #7701
#7701 (
renovate/vitest-eslint-plugin-1.x) was branched against an older tree pinned to vitest 3.2.4 and conflicts withmain. Merging it would downgrade vitest. This PR applies only its intent — the@vitest/eslint-plugin1.3.13 → 1.3.20bump — with no other version changes.Changes
@vitest/eslint-plugin→ 1.3.20. Its@typescript-eslint ^8.46.1dependency resolves as a nested subtree, so the top-leveltypescript-eslintstays at 8.45.0 andviteststays at 3.2.4.prefer-called-exactly-once-withrule; the 5 flagged assertions across 3 test files are collapsed intotoHaveBeenCalledExactlyOnceWith(...)(eslint--fix+ prettier).No
.npmrc/legacy-peer-depsneeded — this is purely on vitest 3.Verification
Local (Node 24):
lint,typecheck,format:checkpass; 373/373 unit tests pass. Integration/e2e not run locally (need auth token / network / Git LFS).Closes #7701.
🤖 Generated with Claude Code