Skip to content

Commit

Permalink
👷 Add Codecov to CI, Smokeshow/Cloudflare has been flaky lately (#1303)
Browse files Browse the repository at this point in the history
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
tiangolo and pre-commit-ci[bot] authored Feb 21, 2025
1 parent ee16ba4 commit 759220d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
10 changes: 10 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions scripts/test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,4 @@ coverage run -m pytest tests
coverage combine
coverage report
coverage html
coverage xml

0 comments on commit 759220d

Please sign in to comment.