Skip to content

Commit 21e444b

Browse files
committed
ci: fix
1 parent 5288d60 commit 21e444b

File tree

1 file changed

+19
-5
lines changed

1 file changed

+19
-5
lines changed

.github/workflows/main.yml

+19-5
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,16 @@ jobs:
6767

6868
- run: yarn compile
6969

70-
- name: Upload compilation results
70+
- name: Upload compilation results (hardhat)
7171
uses: actions/upload-artifact@v4
7272
with:
7373
name: all-artifacts
7474
path: packages/**/artifacts/**
75+
- name: Upload compilation results (forge)
76+
uses: actions/upload-artifact@v4
77+
with:
78+
name: all-artifacts-forge
79+
path: packages/**/out/**
7580

7681
style:
7782
needs: deps
@@ -97,6 +102,11 @@ jobs:
97102

98103
steps:
99104
- uses: actions/checkout@v4
105+
# TODO: right now forge and node tooling coexist, eventually keep only forge action
106+
- uses: foundry-rs/foundry-toolchain@v1
107+
with:
108+
version: nightly
109+
100110
- uses: actions/setup-node@v4
101111
with:
102112
node-version: 20
@@ -108,11 +118,10 @@ jobs:
108118
with:
109119
name: all-artifacts
110120
path: packages/
111-
112-
# TODO: right now forge and node tooling coexist, eventually keep only forge action
113-
- uses: foundry-rs/foundry-toolchain@v1
121+
- uses: actions/download-artifact@v4
114122
with:
115-
version: nightly
123+
name: all-artifacts-forge
124+
path: packages/
116125

117126
- if: contains(needs.changed-files.outputs.modified_files, matrix.dir)
118127
name: Test
@@ -168,6 +177,11 @@ jobs:
168177
with:
169178
path: node_modules
170179
key: ${{ needs.deps.outputs.cache-key }}
180+
# TODO: right now forge and node tooling coexist, eventually keep only forge action
181+
- uses: foundry-rs/foundry-toolchain@v1
182+
with:
183+
version: nightly
184+
171185
- if: contains(needs.changed-files.outputs.modified_files, matrix.dir)
172186
name: Compile contracts
173187
run: |

0 commit comments

Comments
 (0)