Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: improve coverage action #73

Merged
merged 9 commits into from
Jul 4, 2024
Merged

feat: improve coverage action #73

Changes from 4 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
819f0ba
refactor: rename the coverage config
gas1cent Jul 1, 2024
d72936b
refactor: rename coverage action
gas1cent Jul 1, 2024
08537f8
fix: add missing ENVs
gas1cent Jul 1, 2024
7dbdc3e
feat: upgrade actions to v4
gas1cent Jul 1, 2024
46a9317
feat: exclude integration tests from coverage
gas1cent Jul 1, 2024
b76ab0a
feat: add job name
gas1cent Jul 1, 2024
cf57e94
feat: pass `match-path` to `forge coverage`
gas1cent Jul 2, 2024
19e2e1c
feat: improve unit tests filtering in coverage action
gas1cent Jul 2, 2024
e2c7895
refactor: duplicated commands
gas1cent Jul 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions .github/workflows/coverage_check.yml → .github/workflows/coverage.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,11 @@
name: Coverage check on main push
name: Coverage Check

on: [push]

env:
COVERAGE_SENSITIVITY_PERCENT: 1
MAINNET_RPC: ${{ secrets.MAINNET_RPC }}
SEPOLIA_RPC: ${{ secrets.SEPOLIA_RPC }}

gas1cent marked this conversation as resolved.
Show resolved Hide resolved
jobs:
upload-coverage:
Expand Down Expand Up @@ -43,7 +45,7 @@ jobs:
coverage-files: lcovNew.info

- name: Retrieve previous coverage
uses: actions/download-artifact@v2
uses: actions/download-artifact@v4
with:
name: coverage.info
continue-on-error: true
Expand All @@ -66,7 +68,7 @@ jobs:
mv lcovNew.info coverage.info

- name: Upload the new coverage
uses: actions/upload-artifact@v2
uses: actions/upload-artifact@v4
with:
name: coverage.info
path: ./coverage.info
path: ./coverage.info
Loading