Skip to content

Commit e733bc7

Browse files
ricardochlSplaktar
authored andcommitted
fix(ci): fix pnpm installation
The build script (tools/build.mjs) requires pnpm to be available when executing `pnpm install` and `pnpm bazel build` commands. Previously, pnpm was being set up after npm ci and before the build, which caused the build to fail because pnpm was not available in time. Changes: - Move "Setup pnpm" step to run immediately after "Setup Node JS" - Ensures pnpm is available before any build commands are executed
1 parent ea3e52e commit e733bc7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ jobs:
1515
submodules: true
1616
- name: Setup Node JS
1717
uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
18+
- name: Setup pnpm
19+
uses: pnpm/action-setup@v4
20+
with:
21+
version: 10.17.1
1822
- name: Setup Bazel
1923
uses: bazel-contrib/setup-bazel@4fd964a13a440a8aeb0be47350db2fc640f19ca8 # 0.15.0
2024
with:
@@ -23,9 +27,5 @@ jobs:
2327
repository-cache: true
2428
- name: Install Dependencies
2529
run: npm ci
26-
- name: Setup pnpm
27-
uses: pnpm/action-setup@v4
28-
with:
29-
version: 10.17.1
3030
- name: Build Docs
3131
run: npm run build

0 commit comments

Comments
 (0)