Skip to content

test: validate setup-vp Windows stdin-hang fix (PR #91)#1

Draft
fengmk2 wants to merge 3 commits into
mainfrom
test/setup-vp-windows-stdin-fix
Draft

test: validate setup-vp Windows stdin-hang fix (PR #91)#1
fengmk2 wants to merge 3 commits into
mainfrom
test/setup-vp-windows-stdin-fix

Conversation

@fengmk2

@fengmk2 fengmk2 commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Smoke-tests voidzero-dev/setup-vp#91 against this real Tauri + Vite+ build, in particular on windows-latest, where vp builds previously hung for hours until GitHub cancelled them (ref tauri-apps/tauri-action#1308).

Changes:

  • Install vp step now uses voidzero-dev/setup-vp@fix/windows-stdin-hang (the PR branch) instead of @v1.
  • Added a pull_request trigger so this draft PR runs the full build matrix.

Goal: confirm the Windows job completes (no multi-hour hang) end to end with the setup-vp PR.

Draft / do not merge: this is a CI validation PR for the fork.

Smoke-test voidzero-dev/setup-vp PR #91 (close stdin on vp invocations to
avoid the Windows runner hang) against this real Tauri + Vite+ build.

Also add a pull_request trigger so the draft PR runs the full matrix,
including windows-latest, where the hang occurred.
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

fengmk2 added 2 commits June 28, 2026 16:59
Verifies fengmk2/tauri-action#fix/windows-stdin-hang (close child stdin)
against the real build. The previous run reproduced the hang: bundles
finished, then no output until GitHub cancelled the job.
Pin vite-plus to the pkg.pr.new build of voidzero-dev/vite-plus#1973 (the
Windows ps1 stdin-hang fix) via the catalog + overrides, and revert the
workaround actions to stock (setup-vp@v1, tauri-apps/tauri-action@v0). If the
Windows build now completes, the vp-side fix resolves the hang on its own,
without the forked tauri-action stdin-close workaround.
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.

1 participant