Skip to content

Commit e9c44aa

Browse files
shixishclaude
andcommitted
Pin pnpm via packageManager instead of per-workflow version
pnpm/action-setup@v4 now hard-requires an explicit version (no more auto-detection fallback), which broke every workflow here except tests.yml's hardcoded `version: 9` — discovered when the new release.yml's workflow_dispatch dry run failed on "No pnpm version is specified." Adding packageManager to package.json is the single source of truth pnpm/action-setup reads automatically, so drop the now-redundant explicit version from tests.yml too rather than maintaining it in two places. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent cd425b0 commit e9c44aa

2 files changed

Lines changed: 1 addition & 4 deletions

File tree

.github/workflows/tests.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@ jobs:
1515
steps:
1616
- uses: actions/checkout@v4
1717
- uses: pnpm/action-setup@v4
18-
with:
19-
version: 9
2018
- uses: actions/setup-node@v4
2119
with:
2220
node-version: ${{ matrix.node-version }}
@@ -32,8 +30,6 @@ jobs:
3230
steps:
3331
- uses: actions/checkout@v4
3432
- uses: pnpm/action-setup@v4
35-
with:
36-
version: 9
3733
- uses: actions/setup-node@v4
3834
with:
3935
node-version: 22

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
"version": "0.2.0",
44
"description": "TypeScript client library for Diffbot APIs",
55
"type": "module",
6+
"packageManager": "pnpm@10.33.0",
67
"main": "./dist/index.cjs",
78
"module": "./dist/index.js",
89
"types": "./dist/index.d.ts",

0 commit comments

Comments
 (0)