Skip to content

Commit 3896e4b

Browse files
committed
fix(e2e/varlet): install pnpm globally via vp i -g pnpm
Previous fix replaced bootstrap.mjs with explicit vp run commands, but that was unnecessarily complex. Simpler: let `vp i -g pnpm` install pnpm into ~/.vite-plus/bin so scripts/bootstrap.mjs (which spawns `pnpm build` via tinyexec) can resolve it.
1 parent 5f6b37c commit 3896e4b

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

.github/workflows/e2e-test.yml

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -317,16 +317,10 @@ jobs:
317317
- name: varlet
318318
node-version: 22
319319
command: |
320-
# Replace `node scripts/bootstrap.mjs`, which spawns `pnpm build` via
321-
# tinyexec and cannot find pnpm on the CI runner (ENOENT). Run each
322-
# package's build through vp instead, which uses its bundled runtime.
323-
# Skip `-r` to avoid the @varlet/cli <-> @varlet/ui devDependency cycle.
324-
vp run -F '@varlet/shared' -F '@varlet/touch-emulator' -F '@varlet/preset-unocss' -F '@varlet/preset-tailwindcss' -F '@varlet/import-resolver' -F '@varlet/use' -F '@varlet/vite-plugins' build
325-
# tsc reports type errors against @varlet/ui (which isn't compiled yet)
326-
# but still emits JS output — matches upstream bootstrap behavior.
327-
vp run @varlet/cli#build || true
328-
vp run @varlet/icons#build
329-
vp run @varlet/ui#compile
320+
# scripts/bootstrap.mjs spawns `pnpm build` via tinyexec and needs
321+
# pnpm on PATH (not exposed by the vp install itself).
322+
vp i -g pnpm
323+
node scripts/bootstrap.mjs
330324
vp run lint
331325
vp run test:coverage
332326
exclude:

0 commit comments

Comments
 (0)