Skip to content

Commit b902dbb

Browse files
committed
fix(ci): replace bare pnpm calls with vp in fresh-install-tests workflow
setup-vp does not put pnpm on PATH, so the direct pnpm invocations failed with "command not found" (exit 127). vp update and vp dedupe support the same flags and are the correct way to invoke pnpm under vp.
1 parent 628fa25 commit b902dbb

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

.github/workflows/fresh-install-tests.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,6 @@ jobs:
5151
# ranges. This simulates what a user gets when running
5252
# `npm install @blocknote/react` in a fresh project.
5353
# DevDependencies are left at their lockfile versions.
54-
# NB: this uses pnpm directly because vp doesn't expose a `--prod`-only
55-
# update flow; setup-vp installs pnpm on PATH so this still works.
5654
run: |
5755
FILTERS=$(node -e "
5856
const fs = require('fs');
@@ -65,7 +63,7 @@ jobs:
6563
});
6664
")
6765
echo "Updating prod deps for: $FILTERS"
68-
eval pnpm update --prod $FILTERS
66+
eval vp update --prod $FILTERS
6967
7068
- id: dedupe_deps
7169
name: Dedupe transitive dependencies
@@ -78,7 +76,7 @@ jobs:
7876
# Dedupe only rewrites the lockfile — it does NOT modify package.json,
7977
# so the examples' "@blocknote/*": "latest" specs (which is what
8078
# CodeSandbox users see) stay intact.
81-
run: pnpm dedupe
79+
run: vp dedupe
8280

8381
- id: build_packages
8482
name: Build packages

0 commit comments

Comments
 (0)