Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add option to generate test coverage report without codecov #207

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
822a450
add option to generate test coverage report without codecov
zacharyburnett Jun 9, 2024
f95b1fb
update step name
zacharyburnett Jun 10, 2024
2ff2795
update matrix script
zacharyburnett Jun 10, 2024
f9713a8
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jun 10, 2024
c4ba2c9
upload `.coverage.*` file to combine
zacharyburnett Jun 11, 2024
9786a39
list files
zacharyburnett Jun 21, 2024
19c88a4
use runner.os instead of matrix value
zacharyburnett Jun 21, 2024
bcfbd6b
combine and report coverage
zacharyburnett Jun 21, 2024
6740255
undo style formatting
zacharyburnett Jun 21, 2024
a6f284a
set run shell
zacharyburnett Jun 21, 2024
d626757
set job name as `report coverage` to be explicit
zacharyburnett Jun 21, 2024
64f33ca
use conditional ifs instead of continue-of-error
zacharyburnett Jun 24, 2024
80bec60
reorder ls and mv
zacharyburnett Jun 26, 2024
a44231b
find .coverage
zacharyburnett Jun 26, 2024
35381ff
use `find` to find `.coverage`
zacharyburnett Jun 26, 2024
5ec90ff
rename job to indicate that it is reporting overall coverage
zacharyburnett Jun 27, 2024
88ebba9
always run coverage report
zacharyburnett Jun 27, 2024
5146563
search in current directory for coverage file
zacharyburnett Dec 4, 2024
6a8519d
Test a different thing
Cadair Dec 4, 2024
dcb85f1
Apply suggestions from code review
Cadair Dec 4, 2024
e611989
Update .github/workflows/tox.yml
Cadair Dec 4, 2024
0b43061
Update .github/workflows/tox.yml
Cadair Dec 4, 2024
5236380
Update .github/workflows/tox.yml
Cadair Dec 4, 2024
6b908a1
Merge branch 'main' into tox/coverage_github_actions
Cadair Dec 5, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update .github/workflows/tox.yml
Cadair authored Dec 4, 2024
commit e611989127bdf2fef963623556fed42c329ff0f4
9 changes: 1 addition & 8 deletions .github/workflows/tox.yml
Original file line number Diff line number Diff line change
@@ -239,19 +239,12 @@ jobs:
with:
token: ${{ secrets.CODECOV_TOKEN }}

- if: ${{ (success() || failure()) && contains(matrix.coverage, 'github') && matrix.pytest == 'true' }}
name: add run info to coverage filename
run: |
find ${{ github.workspace }} -name .coverage
mv $(find ${{ github.workspace }} -name .coverage) .coverage.${{ github.sha }}-${{ runner.os }}-${{ runner.arch }}-${{ matrix.toxenv }}
shell: sh

- name: Upload coverage data to GitHub
if: ${{ (success() || failure()) && contains(matrix.coverage, 'github') && matrix.pytest == 'true' }}
uses: actions/upload-artifact@v4
with:
name: .coverage.${{ github.sha }}-${{ runner.os }}-${{ runner.arch }}-${{ matrix.toxenv }}
path: .coverage.${{ github.sha }}-${{ runner.os }}-${{ runner.arch }}-${{ matrix.toxenv }}
path: **/.coverage

report_overall_test_coverage:
needs: [ tox ]