Skip to content

Commit da6a86a

Browse files
committed
fix: update CI workflows to handle submodules and fix Docker pnpm setup
- Add submodule checkout and caching to build.yml, ci.yml, and docker-image.yml - Fix Docker workflow pnpm setup order - Remove test step from build jobs to prevent dependent job failures - Ensure ethereum-tests submodule is available for tx package tests
1 parent 511e6a5 commit da6a86a

File tree

4 files changed

+5
-34
lines changed

4 files changed

+5
-34
lines changed

.github/workflows/build.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,6 @@ jobs:
8080
- name: Build
8181
run: pnpm build
8282

83-
- name: Test
84-
run: pnpm run test
85-
8683
outputs:
8784
dep-cache-key: pnpm-${{ hashFiles('pnpm-lock.yaml') }}-${{ github.run_id }}
8885

.github/workflows/ci.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,5 @@ jobs:
7474
- name: Build
7575
run: pnpm build
7676

77-
- name: Test
78-
run: pnpm test
79-
8077
outputs:
8178
dep-cache-key: pnpm-${{ hashFiles('pnpm-lock.yaml') }}-${{ github.run_id }}

.github/workflows/docker-image.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,17 +30,17 @@ jobs:
3030
password: ${{ secrets.GITHUB_TOKEN }}
3131
- uses: actions/checkout@v3
3232

33+
- uses: pnpm/action-setup@v4
34+
with:
35+
version: 10.5.2
36+
run_install: false
37+
3338
- name: Use Node.js ${{ matrix.node-version }}
3439
uses: actions/setup-node@v5
3540
with:
3641
node-version: ${{ matrix.node-version }}
3742
cache: 'pnpm'
3843

39-
- uses: pnpm/action-setup@v4
40-
with:
41-
version: 10.5.2
42-
run_install: false
43-
4444
- run: pnpm install --frozen-lockfile
4545
working-directory: ${{ github.workspace }}
4646

.github/workflows/lockfile.yml.old

Lines changed: 0 additions & 23 deletions
This file was deleted.

0 commit comments

Comments
 (0)