Skip to content

Commit

Permalink
Merge pull request JaGeo#332 from naik-aakash/update_calc_quality
Browse files Browse the repository at this point in the history
Remove redundant coverage uploads and fix coverage reports
  • Loading branch information
JaGeo authored Sep 11, 2024
2 parents a98d1be + e3538ea commit fc1da9b
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,10 @@ jobs:
pytest --cov=lobsterpy --cov-report term-missing --cov-append --splits 6 --group ${{ matrix.split }} -vv --durations-path ./tests/test_data/.pytest-split-durations
- name: Upload coverage
if: matrix.python-version == '3.10'
uses: actions/upload-artifact@v4
with:
name: coverage-${{ matrix.split }}-${{ matrix.python-version }}
name: coverage-${{ matrix.python-version }}-${{ matrix.split }}
include-hidden-files: true
overwrite: false
path: ./.coverage
Expand All @@ -87,11 +88,13 @@ jobs:
- name: Download coverage artifacts
continue-on-error: true
uses: actions/download-artifact@v4
with:
pattern: coverage-3.10-*

- name: Run coverage
continue-on-error: true
run: |
coverage combine coverage*/.coverage-*-3.10
coverage combine coverage*/.coverage*
coverage report --show-missing
- name: Upload coverage reports to Codecov
Expand Down

0 comments on commit fc1da9b

Please sign in to comment.