Skip to content

Commit

Permalink
Fix coverage artifact combination step (#2388)
Browse files Browse the repository at this point in the history
  • Loading branch information
jacobtylerwalls authored Feb 19, 2024
1 parent ddbefc1 commit 6519a0e
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ jobs:
. venv/bin/activate
pytest --cov
- name: Upload coverage artifact
uses: actions/upload-artifact@v4.0.0
uses: actions/upload-artifact@v4.3.1
with:
name: coverage-linux-${{ matrix.python-version }}
path: .coverage
Expand Down Expand Up @@ -179,7 +179,7 @@ jobs:
. venv\\Scripts\\activate
pytest --cov
- name: Upload coverage artifact
uses: actions/upload-artifact@v4.0.0
uses: actions/upload-artifact@v4.3.1
with:
name: coverage-windows-${{ matrix.python-version }}
path: .coverage
Expand Down Expand Up @@ -229,7 +229,7 @@ jobs:
. venv/bin/activate
pytest --cov
- name: Upload coverage artifact
uses: actions/upload-artifact@v4.0.0
uses: actions/upload-artifact@v4.3.1
with:
name: coverage-pypy-${{ matrix.python-version }}
path: .coverage
Expand All @@ -251,10 +251,7 @@ jobs:
- name: Install dependencies
run: pip install -U -r requirements_minimal.txt
- name: Download all coverage artifacts
uses: actions/[email protected]
with:
pattern: coverage-*
merge-multiple: true
uses: actions/[email protected]
- name: Combine Linux coverage results
run: |
coverage combine coverage-linux*/.coverage
Expand Down

0 comments on commit 6519a0e

Please sign in to comment.