Skip to content

Commit

Permalink
fix codecov coverage path
Browse files Browse the repository at this point in the history
  • Loading branch information
ivan committed Jul 26, 2024
1 parent 02b9e05 commit fd28028
Showing 1 changed file with 34 additions and 26 deletions.
60 changes: 34 additions & 26 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,37 +56,45 @@ jobs:
run: make tests-ci

# BUG WITH @v4: https://github.com/actions/upload-artifact/issues/478
- uses: actions/upload-artifact@v3
with:
name: coverage-xml
path: coverage.xml

codecov:
needs:
- lint
- test
runs-on: ubuntu-latest
permissions:
security-events: write
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: "3.12"

- name: Download Artifacts
uses: actions/download-artifact@v4
with:
name: coverage-xml
path: coverage.xml
merge-multiple: true
# - uses: actions/upload-artifact@v3
# with:
# name: coverage-artifact-${{ matrix.python-version }}
# path: coverage-${{ matrix.python-version }}.xml

- name: Upload coverage reports to Codecov
uses: codecov/codecov-action@v4.0.1
uses: codecov/codecov-action@v4
with:
files: coverage.xml
file: ./coverage.xml
token: ${{ secrets.CODECOV_TOKEN }}

# codecov:
# runs-on: ubuntu-latest
# permissions:
# security-events: write
#
# needs:
# - lint
# - test
#
# steps:
# - uses: actions/checkout@v4
# - uses: actions/setup-python@v5
# with:
# python-version: "3.12"
#
# - name: Download Artifacts
# uses: actions/download-artifact@v4
# with:
# name: coverage-xml
# path: coverage.xml
# merge-multiple: true
#
# - name: Upload coverage reports to Codecov
# uses: codecov/[email protected]
# with:
# files: coverage-*.xml
# token: ${{ secrets.CODECOV_TOKEN }}

# code_quality:
# needs: test
# runs-on: ubuntu-latest
Expand Down

0 comments on commit fd28028

Please sign in to comment.