Skip to content

Commit

Permalink
CI: remove codecov on main merge (starkware-libs#1480)
Browse files Browse the repository at this point in the history
* CI: remove codecov on main merge

Currently, codecov reports are not always in adequation with the
code being pushed in a PR.
Disabling on main merge, ensure that maintainers can evaluate
a PR to be mergeable, even if the codecov report is not valid.

* fix: conditioned only the codecov step and not the whole workflow

* fix: make the restriction on main ref instead of event_name
  • Loading branch information
glihm authored Jan 25, 2024
1 parent 374f2bf commit a8258be
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ jobs:
- uses: Swatinem/rust-cache@v2
- run: cargo llvm-cov nextest --all-features --lcov --output-path lcov.info
- uses: codecov/codecov-action@v3
if: github.ref != 'refs/heads/main'
with:
token: ${{ secrets.CODECOV_TOKEN }}
files: lcov.info
Expand Down

0 comments on commit a8258be

Please sign in to comment.