Skip to content

Commit f349092

Browse files
committed
ci: let pnpm/action-setup read packageManager instead of pinning a version
Pinning `version: 9` while package.json also declares `packageManager` makes the action fail with ERR_PNPM_BAD_PM_VERSION. The manifest is the single source of truth.
1 parent cd65974 commit f349092

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ jobs:
2424
# This is a pnpm workspace. npm does not read pnpm-workspace.yaml, so `npm ci`
2525
# silently skips every per-package dependency and the suite fails on imports
2626
# that are correctly declared.
27+
# No `version` here on purpose: package.json declares `packageManager`, and
28+
# specifying both makes the action fail with ERR_PNPM_BAD_PM_VERSION.
2729
- uses: pnpm/action-setup@v4
28-
with:
29-
version: 9
3030

3131
- uses: actions/setup-node@v4
3232
with:

0 commit comments

Comments
 (0)