docs(preact-query): tag 'subscribed' with '@defaultValue' for structured default-value rendering - #11288
Conversation
…red default-value rendering
|
View your CI Pipeline Execution ↗ for commit 6e0f780
☁️ 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 (27)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe change updates Preact query documentation. Source comments now use ChangesPreact documentation updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This change improves how the existing 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description explains the change and motivation, identifies the affected declarations, documents verification with pnpm run generate-docs, completes the checklist, and correctly marks the change as documentation-only with no release impact. Full details: Docstring CoverageExplanation Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (25 skipped: 25 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 📦
|
🎯 Changes
Tags the
subscribedoption's default with TypeDoc's@defaultValuetag, in the three places it's declared (UseBaseQueryOptions,UseInfiniteQueryOptions, and the inlineuseQueriesper-query options).Previously the default was only mentioned as prose inside the description ("Defaults to
true."). TypeDoc renders@defaultValueas a separate, structured "Default Value" section, so this makes the default visually distinct from the description instead of buried in it. Verified withpnpm run generate-docs— the rest of the diff indocs/framework/preact/reference/is line-number drift inDefined in:links from the added JSDoc lines, not content changes.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit
subscribedoption by clearly displaying its default value astrue.