👷 Add Codecov to CI, Smokeshow/Cloudflare has been flaky lately (#1303)
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
This commit is contained in:
parent
ee16ba4dc3
commit
759220d592
10
.github/workflows/test.yml
vendored
10
.github/workflows/test.yml
vendored
@ -74,13 +74,23 @@ jobs:
|
||||
env:
|
||||
COVERAGE_FILE: coverage/.coverage.${{ runner.os }}-py${{ matrix.python-version }}-${{ matrix.pydantic-version }}
|
||||
CONTEXT: ${{ runner.os }}-py${{ matrix.python-version }}
|
||||
# TODO: if removing Smokeshow, and moving only to Codecov, remove this
|
||||
# Upload files before running Codecov, as it generates an extra file coverage/coverage.xml, and that breaks coverage-combine
|
||||
- name: Store coverage files
|
||||
uses: actions/upload-artifact@v4
|
||||
with:
|
||||
name: coverage-${{ matrix.python-version }}-${{ matrix.pydantic-version }}
|
||||
path: coverage
|
||||
include-hidden-files: true
|
||||
- uses: codecov/codecov-action@v5
|
||||
with:
|
||||
fail_ci_if_error: true
|
||||
files: ./coverage.xml
|
||||
name: codecov-umbrella
|
||||
token: ${{ secrets.CODECOV_TOKEN }}
|
||||
verbose: true
|
||||
|
||||
# TODO: if removing Smokeshow, and moving only to Codecov, remove this
|
||||
coverage-combine:
|
||||
needs:
|
||||
- test
|
||||
|
@ -7,3 +7,4 @@ coverage run -m pytest tests
|
||||
coverage combine
|
||||
coverage report
|
||||
coverage html
|
||||
coverage xml
|
||||
|
Loading…
x
Reference in New Issue
Block a user