ci: default tests to latest, add pkg.pr.new dispatch input#74
Merged
Conversation
Remove the `version: [latest, alpha]` matrix from every test job so push/PR/merge_group runs only exercise the latest published Vite+. Add a `workflow_dispatch` input (`pr_version`) for verifying an unreleased Vite+ build on demand. It feeds a workflow-level `VP_PR_VERSION` env var, which the install script checks before `VP_VERSION` and uses to pull the matching build from pkg.pr.new — so a single manual run verifies a candidate across the whole matrix. The value is empty on every other event, which the install script treats as unset and falls back to latest. The build job is pinned to latest (`VP_PR_VERSION: ""`) so a pre-release bundler can't shift dist/ output and fail the "dist is up to date" diff.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates CI to test the default latest Vite+ release on normal events while allowing manual workflow dispatches to verify an unreleased pkg.pr.new Vite+ build via VP_PR_VERSION.
Changes:
- Adds a
workflow_dispatchpr_versioninput and workflow-levelVP_PR_VERSIONenvironment variable. - Removes the
latest/alphaversion matrix from test jobs. - Pins the
buildjob to latest by clearingVP_PR_VERSION, and updates related sfw comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes alpha-channel testing and replaces it with on-demand pkg.pr.new verification triggered manually.
version: [latest, alpha]matrix from every test job. Push / PR / merge_group runs now exercise only the latest published Vite+ (the action's default). Jobs whose only matrix dimension wasversionlost theirstrategy:block; jobs with other dimensions (os,node-version,lockfile) kept theirs.workflow_dispatchinputpr_version(a PR number or commit SHA, e.g.1569) for verifying an unreleased Vite+ build on demand.How pkg.pr.new testing works
pkg.pr.new builds install via
VP_PR_VERSION(PR number / commit SHA), not theversioninput — the install script checksVP_PR_VERSIONbeforeVP_VERSIONand uses it to pull the matching build from pkg.pr.new. The input feeds a workflow-level env var:Since the action spreads
...process.envinto the install, this flows straight through and wins even though the action setsVP_VERSION=latest. On every non-dispatch event the value is empty, which the script treats as unset → latest. So one manual dispatch verifies a candidate build across the entire matrix (install, node-version, cache, exec, sfw) with no source change to the action.Notes
buildjob is pinned to latest (VP_PR_VERSION: "") so a pre-release bundler can't shiftdist/output and fail the "dist is up to date" diff for unrelated reasons.🤖 Generated with Claude Code