Skip to content

Commit

Permalink
disable codecov test analytics to bring back useful coverage reports …
Browse files Browse the repository at this point in the history
  • Loading branch information
fmigneault committed Nov 22, 2024
1 parent 310f4b9 commit f248857
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,15 @@ jobs:
- name: Run Tests
run: make stop ${{ matrix.test-case }}

# disable due to confusing and unhelpful messages created by codecov
# (see )https://github.com/codecov/feedback/issues/304#issuecomment-2492675117)
# - name: Upload test results to Codecov
# if: ${{ !cancelled() && success() && matrix.test-case == 'test-coverage-only' }}
# uses: codecov/test-results-action@v1
# with:
# files: reports/coverage-junit.xml,!./cache
# flags: ${{ matrix.python-version }}
# token: ${{ secrets.CODECOV_TOKEN }}
- name: Upload coverage report
uses: codecov/codecov-action@v2
if: ${{ success() && matrix.test-case == 'test-coverage-only' }}
Expand All @@ -139,13 +148,6 @@ jobs:
files: ./reports/coverage.xml
fail_ci_if_error: true
verbose: true
- name: Upload test results to Codecov
if: ${{ !cancelled() && success() && matrix.test-case == 'test-coverage-only' }}
uses: codecov/test-results-action@v1
with:
files: reports/coverage-junit.xml,!./cache
flags: ${{ matrix.python-version }}
token: ${{ secrets.CODECOV_TOKEN }}

deploy-docker:
needs: tests
Expand Down

0 comments on commit f248857

Please sign in to comment.