Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fix syntax error in pagination example#6125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
basarozcan wants to merge1 commit intoTanStack:main
base:main
Choose a base branch
Loading
frombasarozcan:patch-1

Conversation

@basarozcan
Copy link

@basarozcanbasarozcan commentedNov 7, 2025•
edited by coderabbitaibot
Loading

Adding missing) character.

🎯 Changes

Add missing) character on example while define pagination state in vue

✅ Checklist

  • I have followed the steps in theContributing guide.
  • [] I have tested this code locally withpnpm test:pr.

🚀 Release Impact

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

Summary by CodeRabbit

  • Documentation
    • Corrected a syntax error in the Vue table state guide, fixing a code example in the Controlled State section to ensure accuracy and usability.

Adding missing `)` character.
@changeset-bot
Copy link

⚠️ No Changeset found

Latest commit:a0a9c5d

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

coderabbitaibot commentedNov 7, 2025•
edited
Loading

Walkthrough

Documentation fix correcting a syntax error in a Vue table state example. A missing closing parenthesis in the pagination ref initialization line is added to resolve invalid code syntax.

Changes

Cohort / File(s)Change Summary
Documentation syntax correction
docs/framework/vue/guide/table-state.md		Fixed missing closing parenthesis in controlled state example's pagination ref initialization fromref({ pageIndex: 0, pageSize: 15 } toref({ pageIndex: 0, pageSize: 15 })

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A wayward parenthesis had gone astray,
Lost in the Vue code that guides the way,
But hop and a fix, it's now complete,
Documentation syntax, so neat and sweet! 🐰✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check nameStatusExplanation
Title check✅ PassedThe title clearly and concisely describes the main change: fixing a syntax error in a pagination example by adding a missing closing parenthesis.
Description check✅ PassedThe description follows the required template with all major sections completed. The Changes section explains the modification, the Checklist shows appropriate selections, and Release Impact correctly identifies this as docs-only.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between02c203a anda0a9c5d.

📒 Files selected for processing (1)
  • docs/framework/vue/guide/table-state.md (1 hunks)
🔇 Additional comments (1)
docs/framework/vue/guide/table-state.md (1)

117-117:Syntax fix looks good.

The missing closing parenthesis has been properly added to theref() initialization, allowing developers to copy and run this example without syntax errors.

Thanks for usingCodeRabbit! 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.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@basarozcan
[8]ページ先頭
©2009-2025 Movatter.jp