Skip to content

ci: use voidzero-dev/setup-vp directly - #2463

Open
lachlancollins wants to merge 2 commits into
mainfrom
setup-vp
Open

ci: use voidzero-dev/setup-vp directly#2463
lachlancollins wants to merge 2 commits into
mainfrom
setup-vp

Conversation

@lachlancollins

@lachlancollins lachlancollins commented Nov 2, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

🎯 Changes

Replaces our custom setup step (which is basically just a wrapper for setup-vp now) with the direct setup-vp action.

See also: TanStack/config#429

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I have tested code changes locally with pnpm test:pr, or these tests do not apply to this pull request.
  • I fully understand the code in this pull request, including any code generated with AI assistance.

🚀 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

  • Chores
    • Updated automated checks, preview builds, documentation generation, and release workflows to use Vite+ commands.
    • Preserved the existing conditions for documentation generation and preview publishing.

@lachlancollins
lachlancollins requested a review from a team as a code owner November 2, 2026 22:01
@changeset-bot

changeset-bot Bot commented Nov 2, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: af8e5fc

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

coderabbitai Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between eba4084 and af8e5fc.

Files selected for processing: 4
  • .github/workflows/autofix.yml
  • .github/workflows/pr.yml
  • .github/workflows/release.yml
  • .node-version

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.

Walkthrough

The autofix, pull request, and release workflows replace the shared setup action and selected pnpm commands with the pinned Vite+ setup action and vp or vpx commands.

Changes: Workflow Toolchain Migration

Layer / File(s) Summary
Automation and pull request workflows
.github/workflows/autofix.yml, .github/workflows/pr.yml		 These workflows use the pinned Vite+ setup action. Formatting, documentation generation, tests, builds, and preview publishing now use Vite+ commands. The documentation-generation condition and preview publish arguments remain unchanged.
Release workflow
.github/workflows/release.yml		 The release workflow uses the pinned Vite+ setup action. CI tests and Changesets versioning and publishing now use vp run.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Possibly related PRs

  • TanStack/form#2455: Updates the pinned setup action in the same workflows that this pull request changes.
  • TanStack/form#2454: Updates the pinned setup action in the same workflows that this pull request changes.

Suggested reviewers: crutchcorn

Merge Risk: Minimal · up to af8e5

No actionable merge-blocking risk remains in the workflow migration.

Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly summarizes the main change: using the voidzero-dev/setup-vp action directly.
Description check Passed The description covers the change and motivation, completes the checklist, and identifies the change as CI/dev-only with no release impact.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@nx-cloud

nx-cloud Bot commented Nov 2, 2026 •
edited
Loading

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit af8e5fc

Command Status Duration Result
nx run-many --target=build --exclude=examples/** ✅ Succeeded 39s View ↗

☁️ Nx Cloud last updated this comment at 2026-11-02 22:09:35 UTC

@pkg-pr-new

pkg-pr-new Bot commented Nov 2, 2026

Copy link
Copy Markdown
More templates

@tanstack/angular-form

npm i https://pkg.pr.new/@tanstack/angular-form@2463

@tanstack/form-core

npm i https://pkg.pr.new/@tanstack/form-core@2463

@tanstack/form-devtools

npm i https://pkg.pr.new/@tanstack/form-devtools@2463

@tanstack/lit-form

npm i https://pkg.pr.new/@tanstack/lit-form@2463

@tanstack/preact-form

npm i https://pkg.pr.new/@tanstack/preact-form@2463

@tanstack/react-form

npm i https://pkg.pr.new/@tanstack/react-form@2463

@tanstack/react-form-devtools

npm i https://pkg.pr.new/@tanstack/react-form-devtools@2463

@tanstack/react-form-nextjs

npm i https://pkg.pr.new/@tanstack/react-form-nextjs@2463

@tanstack/react-form-remix

npm i https://pkg.pr.new/@tanstack/react-form-remix@2463

@tanstack/react-form-start

npm i https://pkg.pr.new/@tanstack/react-form-start@2463

@tanstack/solid-form

npm i https://pkg.pr.new/@tanstack/solid-form@2463

@tanstack/solid-form-devtools

npm i https://pkg.pr.new/@tanstack/solid-form-devtools@2463

@tanstack/svelte-form

npm i https://pkg.pr.new/@tanstack/svelte-form@2463

@tanstack/vue-form

npm i https://pkg.pr.new/@tanstack/vue-form@2463

commit: af8e5fc

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

🚀 Changeset Version Preview

3 package(s) bumped directly, 11 bumped as dependents.

🟩 Patch bumps

Package Version Reason
@tanstack/form-devtools 0.2.34 → 0.2.35 Changeset
@tanstack/solid-form 1.33.5 → 1.33.6 Changeset
@tanstack/solid-form-devtools 0.2.34 → 0.2.35 Changeset
@tanstack/angular-form 1.33.5 → 1.33.6 Dependent
@tanstack/form-core 1.33.5 → 1.33.6 Dependent
@tanstack/lit-form 1.25.5 → 1.25.6 Dependent
@tanstack/preact-form 1.30.5 → 1.30.6 Dependent
@tanstack/react-form 1.33.5 → 1.33.6 Dependent
@tanstack/react-form-devtools 0.2.34 → 0.2.35 Dependent
@tanstack/react-form-nextjs 1.33.5 → 1.33.6 Dependent
@tanstack/react-form-remix 1.33.5 → 1.33.6 Dependent
@tanstack/react-form-start 1.33.5 → 1.33.6 Dependent
@tanstack/svelte-form 1.33.5 → 1.33.6 Dependent
@tanstack/vue-form 1.33.5 → 1.33.6 Dependent

This branch has not been deployed

No deployments
Sign up for free to 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

@lachlancollins