Skip to content

Conversation

@Olexandr88
Copy link
Contributor

@Olexandr88 Olexandr88 commented Oct 18, 2025

🎯 Changes

βœ… Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested this code locally with pnpm run test:pr.

πŸš€ Release Impact

  • This change affects published code, and I have generated a changeset.
  • This change is docs/CI/dev-only (no release).

Summary by CodeRabbit

  • Documentation
    • Updated the README's GitHub link to point to the repository's stargazers page, clarifying where to view project stargazers.

@changeset-bot
Copy link

changeset-bot bot commented Oct 18, 2025

⚠️ No Changeset found

Latest commit: 131aeee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 18, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The README.md badge anchor href was changed to point to the repository's stargazers page instead of the repository root; no other documentation or code was modified.

Changes

Cohort / File(s) Change Summary
Documentation link update
README.md
Updated GitHub repository hyperlink destination from https://github.com/TanStack/query/ to https://github.com/TanStack/query/stargazers

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • Readme fixΒ #9652 β€” README badge/link markup changes (anchor/href and badge formatting) closely related to this README link update.
  • docs: README UpdatesΒ #9640 β€” Updates to the README badge area/links that overlap with this change.

Poem

I hopped through lines both short and sweet,
A tiny change beneath my feet,
From repo root to stars I steer,
A gentle click, the path made clear πŸ‡βœ¨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description follows the required template structure with all three sections present (Changes, Checklist, and Release Impact). However, the critical "Changes" section is completely emptyβ€”it only contains the HTML comment placeholder with no description of what was changed or the motivation behind the change. While the template structure is correct, this section should explain that the link was updated from the repository root to the stargazers page and why this change was made. The Checklist and Release Impact sections are also not filled out (items remain unchecked), further indicating incomplete submission of the PR description. The description is largely incomplete as it lacks the core information about the actual changes being proposed. The author should provide a detailed description in the "Changes" section explaining what link was updated (the stars badge link in README.md) and why (to direct users to the stargazers page). Additionally, the Checklist items should be reviewed and appropriately checked or unchecked based on whether the contributing guide was followed and tests were run. If this is a docs-only change with no release impact, that should be clearly indicated in the Release Impact section.
βœ… Passed checks (2 passed)
Check name Status Explanation
Title Check βœ… Passed The PR title "fix(docs): add a link to the stars badge" directly and clearly describes the main change in the pull request. According to the raw summary, the changeset updates a GitHub link in README.md from the repository root to the stargazers page, which aligns with the stated purpose of "adding a link to the stars badge." The title is concise, specific, and follows conventional commit messaging with a proper scope (docs). A teammate reviewing the history would understand that this PR modifies documentation by linking the stars badge to the stargazers page.
Docstring Coverage βœ… Passed No functions found in the changes. Docstring coverage check skipped.

πŸ“œ Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between e7bb3cf and 131aeee.

πŸ“’ Files selected for processing (1)
  • README.md (1 hunks)

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.

@TkDodo TkDodo merged commit c578b3c into TanStack:main Nov 1, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants