Skip to content

Commit 021c475

Browse files
committed
fix(ci): correct yaml syntax for pnpm install step
1 parent 550e03c commit 021c475

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

.github/workflows/publish.yml

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ jobs:
2626
- name: Set up Node.js
2727
uses: actions/setup-node@v4.0.3 # Specific version
2828
with:
29-
- name: Install pnpm globally
30-
run: npm install -g pnpm
31-
3229
node-version: '22' # Use latest LTS
3330
cache: 'pnpm' # Use pnpm cache
3431

32+
- name: Install pnpm globally
33+
run: npm install -g pnpm
34+
3535
- name: Install dependencies
3636
run: pnpm install --frozen-lockfile # Use pnpm install
3737

@@ -62,9 +62,6 @@ jobs:
6262
version: ${{ steps.get_version.outputs.version }}
6363
artifact_name: build-artifacts-archive # Consistent name for upload/download
6464
archive_filename: ${{ steps.archive_build.outputs.archive_name }} # Actual .tar.gz filename
65-
- name: Install pnpm globally
66-
run: npm install -g pnpm
67-
6865
steps:
6966
- name: Checkout repository
7067
uses: actions/checkout@v4.1.7 # Specific version
@@ -75,6 +72,9 @@ jobs:
7572
node-version: '22' # Use latest LTS
7673
cache: 'pnpm' # Use pnpm cache
7774

75+
- name: Install pnpm globally
76+
run: npm install -g pnpm
77+
7878
- name: Install dependencies
7979
run: pnpm install --frozen-lockfile # Use pnpm install
8080

@@ -120,9 +120,6 @@ jobs:
120120
uses: actions/download-artifact@v4.1.8 # Specific version
121121
with:
122122
name: ${{ needs.build.outputs.artifact_name }} # Use consistent artifact name
123-
- name: Install pnpm globally
124-
run: npm install -g pnpm
125-
126123
path: .
127124

128125
- name: Extract build artifacts
@@ -135,6 +132,9 @@ jobs:
135132
registry-url: 'https://registry.npmjs.org/'
136133
cache: 'pnpm' # Use pnpm cache
137134

135+
- name: Install pnpm globally
136+
run: npm install -g pnpm
137+
138138
# Install only production dependencies before publishing (optional but good practice)
139139
# - name: Install production dependencies
140140
# run: npm ci --omit=dev

memory-bank/activeContext.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
## 2. Recent Changes/Decisions
1111

12-
- **GitHub Actions:** Added explicit `npm install -g pnpm` step to `validate`, `build`, and `publish-npm` jobs in `.github/workflows/publish.yml` to fix potential `pnpm` not found errors.
12+
- **GitHub Actions:** Corrected YAML syntax in `.github/workflows/publish.yml` after previous failed edits. Ensured `npm install -g pnpm` step is correctly placed after `setup-node` in `validate`, `build`, and `publish-npm` jobs.
1313

1414
- Removed `edit_file` tool and related files (`editFile.ts`, `editFileUtils.ts`, `editFile.test.ts`).
1515
- Added `apply_diff` tool (`applyDiff.ts`, `applyDiffUtils.ts`, `applyDiffSchema.ts`, `applyDiff.test.ts`).

0 commit comments

Comments
 (0)