Skip to content

Commit

Permalink
ci: fix isntall of forge
Browse files Browse the repository at this point in the history
  • Loading branch information
sripwoud committed Jul 3, 2024
1 parent 5288d60 commit f8c205d
Showing 1 changed file with 14 additions and 5 deletions.
19 changes: 14 additions & 5 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,11 +67,16 @@ jobs:

- run: yarn compile

- name: Upload compilation results
- name: Upload compilation results (hardhat)
uses: actions/upload-artifact@v4
with:
name: all-artifacts
path: packages/**/artifacts/**
- name: Upload compilation results (forge)
uses: actions/upload-artifact@v4
with:
name: all-artifacts-forge
path: packages/**/out/**

style:
needs: deps
Expand Down Expand Up @@ -108,11 +113,10 @@ jobs:
with:
name: all-artifacts
path: packages/

# TODO: right now forge and node tooling coexist, eventually keep only forge action
- uses: foundry-rs/foundry-toolchain@v1
- uses: actions/download-artifact@v4
with:
version: nightly
name: all-artifacts-forge
path: packages/

- if: contains(needs.changed-files.outputs.modified_files, matrix.dir)
name: Test
Expand Down Expand Up @@ -168,6 +172,11 @@ jobs:
with:
path: node_modules
key: ${{ needs.deps.outputs.cache-key }}
# TODO: right now forge and node tooling coexist, eventually keep only forge action
- uses: foundry-rs/foundry-toolchain@v1
with:
version: nightly

- if: contains(needs.changed-files.outputs.modified_files, matrix.dir)
name: Compile contracts
run: |
Expand Down

0 comments on commit f8c205d

Please sign in to comment.