chore(*): declare 'eslint-plugin-react-hooks' in the react packages that use it instead of the root - #11725
Conversation
…hat use it instead of the root
|
View your CI Pipeline Execution ↗ for commit c7bb2e8
☁️ 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 configurationConfiguration used: Repository: TanStack/query/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review. 📝 WalkthroughWalkthroughThe root manifest no longer lists ChangesDevelopment dependency scope
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~5 minutes Change: Other Suggested reviewers: Merge Risk: ⚪ Minimal · up to The checked-in lint and build workflows retain the plugin where they need it, and the lockfile matches the dependency changes. No supported workflow failure was identified. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 5 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
eslint-plugin-react-hooks(^7.1.1) from the rootdevDependenciesto thedevDependenciesofreact-query,react-query-devtools,react-query-next-experimental, andreact-query-persist-client, which are the only packages whoseeslint.config.jsimports it.pnpm-lock.yaml(deduped so the plugin keeps a single peer variant; no new packages or versions).✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit