Skip to content

Commit 488642d

Browse files
committed
fix: remove npm cache from CTC workflows to support pnpm/yarn repos
ctcOpen.yml and ctcClose.yml set actions/setup-node `cache: npm`, which requires an npm lockfile to compute its cache key. Consumer repos on pnpm or yarn have no package-lock.json, so setup-node fails with "Dependencies lock file is not found" and the CTC job dies before it ever runs. The only npm operation in these jobs is the one-off global install of @salesforce/change-case-management, which the cache does not key on, so it provides no benefit here. Dropping `cache: npm` lets the CTC legs run for npm, yarn, and pnpm consumers alike.
1 parent b1f7047 commit 488642d

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/ctcClose.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ jobs:
2020
- uses: actions/setup-node@v4
2121
with:
2222
node-version: lts/*
23-
cache: npm
23+
# No `cache: npm`: it requires an npm lockfile and so hard-fails on
24+
# pnpm/yarn consumer repos; the CTC CLI below is a one-off global install.
2425
- run: npm install -g @salesforce/change-case-management --omit=dev
2526
- id: ctc
2627
run: |

.github/workflows/ctcOpen.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ jobs:
2020
- uses: actions/setup-node@v4
2121
with:
2222
node-version: lts/*
23-
cache: npm
23+
# No `cache: npm`: it requires an npm lockfile and so hard-fails on
24+
# pnpm/yarn consumer repos; the CTC CLI below is a one-off global install.
2425

2526
- run: npm install -g @salesforce/change-case-management --omit=dev
2627

0 commit comments

Comments
 (0)