docs(preact-query): backtick bare code-symbol references in JSDoc ('QueryClient', 'suspense', 'throwOnError') - #11290
Conversation
|
View your CI Pipeline Execution ↗ for commit ab40c94
☁️ Nx Cloud last updated this comment at |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
🚧 Files skipped from review as they are similar to previous changes (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe PR updates Preact ChangesPreact documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This is a localized documentation-only change that does not alter runtime behavior or published code; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 14 files. (2 skipped: 2 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
size-limit report 📦
|
… 'suspense'/'throwOnError' in QueryErrorResetBoundary
🎯 Changes
Wraps bare code-symbol references in backticks across the package's JSDoc, found via a systematic audit against the codebase's existing backtick convention for types, fields, and literal values:
QueryClienttype name was bare in the@param queryClientdescription used throughout the package ("Use this to use/provide a custom QueryClient...") and inQueryClientProviderProps'sclient/childrenfield docs. This wording was introduced verbatim in docs(preact-query): document functions and components with JSDoc for generated reference docs #11204 and repeated in every hook that accepts an optionalqueryClientargument — a single consistent fix across 13 source files (21 occurrences).suspenseandthrowOnError— actual option field names fromQueryObserverOptions— were bolded (**suspense**/**throwOnError**) instead of backticked inQueryErrorResetBoundary's summary.Every other code-symbol reference in the package's JSDoc (types, fields, literals, package names) is already backtick-wrapped or
{@link}-linked — these were the only gaps. Verified withpnpm run generate-docsandpnpm --filter @tanstack/preact-query run test:types.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact