Skip to content

test(cli): stabilize global snap tests against pnpm v11#1528

Merged
fengmk2 merged 2 commits intomainfrom
fix-flaky-snap-tests
May 7, 2026
Merged

test(cli): stabilize global snap tests against pnpm v11#1528
fengmk2 merged 2 commits intomainfrom
fix-flaky-snap-tests

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented May 7, 2026

Five fixtures that ran vp install or vp migrate had no
packageManager set, so vite_install fell through to its latest
default (now pnpm v11). pnpm v11 fails the install with
ERR_PNPM_IGNORED_BUILDS for transitive esbuild/sharp build scripts
where v10 only warned. Pin each fixture to pnpm@10.33.2 to match the
repo's own packageManager.

Separately, create-framework-shim-astro's install step also tripped
the strict-CI builds check on a clean runner; pass --ignore-scripts
so pnpm doesn't try to run them at all.

Five fixtures that ran `vp install` or `vp migrate` had no
`packageManager` set, so vite_install fell through to its `latest`
default (now pnpm v11). pnpm v11 fails the install with
`ERR_PNPM_IGNORED_BUILDS` for transitive esbuild/sharp build scripts
where v10 only warned. Pin each fixture to `pnpm@10.33.2` to match the
repo's own packageManager.

Separately, `create-framework-shim-astro`'s install step also tripped
the strict-CI builds check on a clean runner; pass `--ignore-scripts`
so pnpm doesn't try to run them at all.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 7, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit 709ceb1
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69fc95d4f101ff0007f5191a

@fengmk2 fengmk2 marked this pull request as ready for review May 7, 2026 13:28
Copy link
Copy Markdown
Member Author

fengmk2 commented May 7, 2026


How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fengmk2 fengmk2 requested review from branchseer and cpojer May 7, 2026 13:28
@fengmk2 fengmk2 self-assigned this May 7, 2026
Same root cause as 7d65d0c: missing `packageManager` makes
vite_install fall through to pnpm@latest (now v11), which fails the
install on transitive build scripts.
@fengmk2 fengmk2 merged commit d248980 into main May 7, 2026
39 checks passed
@fengmk2 fengmk2 deleted the fix-flaky-snap-tests branch May 7, 2026 13:49
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.

2 participants