Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughTest files in the vue-query package have been updated to use the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
View your CI Pipeline Execution ↗ for commit 2b56e27
☁️ 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. |
size-limit report 📦
|
🎯 Changes
Replace hardcoded query keys with the
queryKey()utility from@tanstack/query-test-utilsacross vue-query test files, following the pattern used in other adapters (react-query, solid-query, etc.).Files changed
Runtime tests (
.test.ts):queryOptions.test.tsinfiniteQueryOptions.test.tsusePrefetchQuery.test.tsusePrefetchInfiniteQuery.test.tsType tests (
.test-d.ts):queryOptions.test-d.tsusePrefetchQuery.test-d.tsusePrefetchInfiniteQuery.test-d.tsuseQueries.test-d.tsuseQuery.test-d.tsExcluded
queryCache.test.ts: keeps hardcoded['foo', ref('bar')]because the test's purpose is to verify ref unwrapping (the fixed values are assertion targets, not placeholders).✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit