Skip to content

Commit

Permalink
ci: fix
Browse files Browse the repository at this point in the history
  • Loading branch information
sripwoud committed Jul 3, 2024
1 parent 5288d60 commit 21e444b
Showing 1 changed file with 19 additions and 5 deletions.
24 changes: 19 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 All @@ -97,6 +102,11 @@ jobs:

steps:
- uses: actions/checkout@v4
# TODO: right now forge and node tooling coexist, eventually keep only forge action
- uses: foundry-rs/foundry-toolchain@v1
with:
version: nightly

- uses: actions/setup-node@v4
with:
node-version: 20
Expand All @@ -108,11 +118,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 +177,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 21e444b

Please sign in to comment.