Skip to content

ci: use setup-vp for JavaScript tooling - #3245

Open
afonsojramos wants to merge 2 commits into
mainfrom
chore/setup-vp-action
Open

ci: use setup-vp for JavaScript tooling#3245
afonsojramos wants to merge 2 commits into
mainfrom
chore/setup-vp-action

Conversation

@afonsojramos

@afonsojramos afonsojramos commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

CI now lets the official Vite+ action own Vite+, Node.js, pnpm, dependency installation, and pnpm-store caching instead of composing those pieces ourselves. This isolates whether Vite+ materially simplifies Gitify's workflow setup before we decide on the larger toolchain direction.

setup-vp resolves the existing project pins: Vite+ 0.2.9 from package.json, Node.js 24.20.0, and pnpm 11.24.0 from packageManager. Jobs that need dependencies use the action's default vp install; Renovate config validation keeps installation disabled.

The action uses the pinned package manager internally but does not expose a bare pnpm executable to later workflow steps. Those steps now use Vite+'s supported command surface instead: vp run for package scripts, vp exec for project binaries, and vp dlx for the Renovate validator.

Related: #3244

Validation

  • This PR's CI passed environment detection, cache restoration, installation, lint, TypeScript, unit tests, visual regression, SonarQube analysis, and packaging on Linux, macOS, and Windows.
  • The repository's pinned actionlint 1.7.12 accepts every workflow.
  • Renovate config validation passes through vp dlx --package renovate renovate-config-validator.
  • zizmor 1.30.0 reports no findings from the new external action usage; its remaining findings are pre-existing repository advisories.

@afonsojramos
afonsojramos requested a review from setchy as a code owner September 1, 2026 20:41
@github-actions github-actions Bot added the build Build, action or package manager changes label Sep 1, 2026
setup-vp exposes the Vite+ CLI but not the underlying pnpm binary to
subsequent workflow steps. Use Vite+ to run scripts and binaries, and
validate Renovate without a global package-manager install.
@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@afonsojramos

Copy link
Copy Markdown
Member Author

@setchy I think that the issue might have been that we only partially adopted vp. With full adoption, we might have no more issues.

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

Labels

build Build, action or package manager changes

Development

Successfully merging this pull request may close these issues.

1 participant