Skip to content

Commit 176a887

Browse files
committed
update to combine files in coverage report
1 parent dab35b2 commit 176a887

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

.github/workflows/build_tests.yml

+7-7
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ jobs:
3636
run: |
3737
python setup.py bdist_wheel
3838
ls dist/*
39-
- name: Save wheel
39+
- name: Upload wheels
4040
uses: actions/upload-artifact@v4
4141
with:
4242
name: pecos_${{ matrix.python-version }}_${{ matrix.os }}.whl
@@ -87,7 +87,7 @@ jobs:
8787
pip install -r requirements.txt
8888
python -m pip install -e .
8989
- name: Run Tests
90-
run: |
90+
run: |
9191
coverage erase
9292
coverage run --context=${{ matrix.os }}.py${{ matrix.python-version }} --source=pecos --omit="*/tests/*" -m pytest --doctest-modules --doctest-glob="*.rst" pecos
9393
coverage run --context=${{ matrix.os }}.py${{ matrix.python-version }} --source=pecos --omit="*/tests/*" --append -m pytest --doctest-glob="*.rst" documentation
@@ -96,7 +96,7 @@ jobs:
9696
- name: Save coverage
9797
uses: actions/upload-artifact@v4
9898
with:
99-
name: coverage
99+
name: .coverage.${{ matrix.python-version }}.${{ matrix.os }}
100100
path: .coverage.${{ matrix.python-version }}.${{ matrix.os }}
101101

102102
combine_reports:
@@ -117,7 +117,9 @@ jobs:
117117
- name: Download coverage artifacts from test matrix
118118
uses: actions/download-artifact@v4
119119
with:
120-
name: coverage
120+
pattern: .coverage.*.ubuntu-latest # coverage from other OS cause problems
121+
- name: Setup coverage and combine reports
122+
run: coverage combine .coverage.*.ubuntu-latest
121123
- name: Create coverage report
122124
run: |
123125
echo "[paths]" > .coveragerc
@@ -142,9 +144,7 @@ jobs:
142144
with:
143145
name: coverage-html
144146
path: htmlcov
145-
146147
- name: Push to coveralls
147-
run: |
148-
coveralls --service=github
148+
run: coveralls --service=github --rcfile=.coveragerc
149149
env:
150150
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)