Skip to content

Commit

Permalink
refactor: remove and modify some small details
Browse files Browse the repository at this point in the history
  • Loading branch information
nadir-akhtar committed Feb 7, 2025
1 parent 3ddc338 commit 774ff7e
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 18 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/forge-test-intense.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ on:
workflow_dispatch:
push:
branches:
- master
- mainnet
- testnet-holesky
- dev
Expand All @@ -30,13 +29,13 @@ jobs:
submodules: recursive

# Install the Foundry toolchain.
- name: "Install Foundry"
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
with:
version: stable

# Build the project and display contract sizes.
- name: "Forge Build"
- name: Forge Build
run: |
forge --version
forge build --sizes
Expand Down
25 changes: 10 additions & 15 deletions .github/workflows/foundry.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,8 @@ on:
workflow_dispatch:
push:
branches:
- master
- mainnet
- testnet-goerli
- testnet-holesky
- dev
pull_request:

Expand All @@ -24,39 +23,37 @@ jobs:
test:
name: Test
runs-on: ubuntu-latest
strategy:
fail-fast: true
steps:
# Check out repository with all submodules for complete codebase access.
- uses: actions/checkout@v4
with:
submodules: recursive

# Install the Foundry toolchain.
- name: "Install Foundry"
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
with:
version: stable

# Run Forge's formatting checker to ensure consistent code style.
- name: "Forge Fmt"
- name: Forge Fmt
run: |
forge fmt --check
id: fmt

# Build the project and display contract sizes.
- name: "Forge Build"
- name: Forge Build
run: |
forge --version
forge build --sizes
id: build

# Run local tests (unit and integration).
- name: "Forge Test (Local)"
- name: Forge Test (Local)
run: forge test -vvv

# Run integration tests using a mainnet fork.
- name: "Forge Test Integration (Fork)"
- name: Forge Test Integration (Fork)
run: FOUNDRY_PROFILE=forktest forge test --match-contract Integration -vvv

# -----------------------------------------------------------------------
Expand All @@ -66,16 +63,14 @@ jobs:
run-coverage:
name: Coverage
runs-on: ubuntu-latest
strategy:
fail-fast: true
steps:
# Check out repository with all submodules for complete codebase access.
- uses: actions/checkout@v4
with:
submodules: recursive

# Install the Foundry toolchain.
- name: "Install Foundry"
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
with:
version: stable
Expand All @@ -87,7 +82,7 @@ jobs:
id: lcov

# Build the project and display contract sizes.
- name: "Forge Build"
- name: Forge Build
run: |
forge --version
forge build --sizes
Expand All @@ -107,7 +102,7 @@ jobs:
path: report/*

# Check coverage threshold after uploading report
- name: Check Coverage Threshold
- name: Check Coverage Threshold for >=90%
run: |
LINES_PCT=$(lcov --summary lcov.info | grep "lines" | cut -d ':' -f 2 | cut -d '%' -f 1 | tr -d '[:space:]')
FUNCTIONS_PCT=$(lcov --summary lcov.info | grep "functions" | cut -d ':' -f 2 | cut -d '%' -f 1 | tr -d '[:space:]')
Expand Down Expand Up @@ -145,7 +140,7 @@ jobs:
submodules: recursive

# Install the Foundry toolchain.
- name: "Install Foundry"
- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
with:
version: stable
Expand Down

0 comments on commit 774ff7e

Please sign in to comment.