Skip to content

fix(docs): remove duplicate code block in query integration guide#7190

Open
CBell045 wants to merge 2 commits intoTanStack:mainfrom
CBell045:fix/docs-query-framework-duplicate
Open

fix(docs): remove duplicate code block in query integration guide#7190
CBell045 wants to merge 2 commits intoTanStack:mainfrom
CBell045:fix/docs-query-framework-duplicate

Conversation

@CBell045
Copy link
Copy Markdown

@CBell045 CBell045 commented Apr 15, 2026

Summary by CodeRabbit

  • Documentation
    • Removed an unscoped example and left the framework-scoped examples in the "Using useSuspenseQuery vs useQuery" section to reduce duplication and ensure framework-specific samples are shown consistently.

@github-actions github-actions bot added the documentation Everything documentation related label Apr 15, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 73a90d30-9ece-4237-b7ec-6ed8b544df8c

📥 Commits

Reviewing files that changed from the base of the PR and between 6346d81 and 848ee03.

📒 Files selected for processing (1)
  • docs/router/integrations/query.md
💤 Files with no reviewable changes (1)
  • docs/router/integrations/query.md

📝 Walkthrough

Walkthrough

Removed an unscoped tsx code block from the "Using useSuspenseQuery vs useQuery" subsection in docs/router/integrations/query.md, leaving only the framework-scoped React examples; no other text or code samples were changed.

Changes

Cohort / File(s) Summary
Documentation
docs/router/integrations/query.md
Deleted an unscoped tsx example block showing useSuspenseQuery/useQuery; retained and relied on the existing framework-scoped (<!-- ::start:framework -->) React examples. No other edits made.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I nudged a code block, gentle as a breeze,
Tabs now whisper clearly, React in its place,
Docs hop along tidy, no clutter to tease,
Examples snug and cozy, neat as a lace. 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing a duplicate code block from the query integration guide documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/router/integrations/query.md`:
- Line 82: The duplicate heading "# React" triggers markdownlint MD024; wrap the
repeated framework heading block by inserting a localized markdownlint
disable/enable pair around the duplicate heading (disable MD024 before the
repeated "# React" and re-enable after) so the heading text can remain unchanged
while silencing the duplicate-heading lint warning for that block.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: dae43d94-aa2a-4efc-b4de-bc96521bc697

📥 Commits

Reviewing files that changed from the base of the PR and between e30814d and 6346d81.

📒 Files selected for processing (1)
  • docs/router/integrations/query.md

@schiller-manuel
Copy link
Copy Markdown
Contributor

this is not the correct fix. dont add solid. remove the duplicate code block.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants