Repository navigation
chore(eslint-plugin-query): move '@typescript-eslint/rule-tester' to the pnpm catalog - #11822
Conversation
|
|
View your CI Pipeline Execution ↗ for commit 26d3a23
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe workspace catalog now defines the ChangesDependency catalog update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The development dependency remains at the same resolved version, so this change presents no material user-facing risk and is ready to merge. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
size-limit report 📦
|
🎯 Changes
Moves
@typescript-eslint/rule-tester(adevDependencyofeslint-plugin-query) to the pnpm catalog, next to@typescript-eslint/parser:typescript-eslintpackages are released together under the same version, so keeping both in the catalog lets them be updated in one place. The resolved version stays8.71.0.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit