Skip to content

Commit aa0c051

Browse files
adamsofferclaude
andcommitted
Point the ecosystem URL check at pnpm and the right paths
The job has been failing on every push since the redesign, for two reasons both introduced by it. It runs `npm ci` behind `cache: npm`, which needs the package-lock.json deleted when the lockfiles were reconciled — the repo pins pnpm via packageManager and ships pnpm-lock.yaml. And it triggers on data/ecosystem.json, which is the catalogue's old shape; the data moved to markdown in content/ecosystem, so the guard had quietly stopped firing on the files it exists to check. The script itself was fine: it already reads content/ecosystem, and passes locally at 12 URLs checked, 0 failed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent e63f2f9 commit aa0c051

1 file changed

Lines changed: 11 additions & 4 deletions

File tree

.github/workflows/check-ecosystem-urls.yml

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
name: Check Ecosystem URLs
22

33
on:
4-
# Run on PRs that touch ecosystem data
4+
# Run on PRs that touch ecosystem data. The catalogue is markdown in
5+
# content/ecosystem — data/ecosystem.json was its old shape and no longer
6+
# exists, so this trigger had stopped firing on the files it is meant to guard.
57
pull_request:
68
paths:
7-
- "data/ecosystem.json"
9+
- "content/ecosystem/**"
810
- "public/ecosystem/**"
911

1012
# Weekly check — Mondays at 9am UTC
@@ -20,13 +22,18 @@ jobs:
2022
steps:
2123
- uses: actions/checkout@v4
2224

25+
# pnpm, not npm: the repo pins it via package.json's packageManager field
26+
# and ships pnpm-lock.yaml. This step must precede setup-node so that
27+
# `cache: pnpm` can find the store.
28+
- uses: pnpm/action-setup@v4
29+
2330
- uses: actions/setup-node@v4
2431
with:
2532
node-version: 20
26-
cache: npm
33+
cache: pnpm
2734

2835
- name: Install dependencies
29-
run: npm ci
36+
run: pnpm install --frozen-lockfile
3037

3138
- name: Check ecosystem URLs
3239
run: node scripts/check-ecosystem-urls.mjs

0 commit comments

Comments
 (0)