Skip to content

Commit a1fdc57

Browse files
authored
👷 Revert "Add Codecov to CI, Smokeshow/Cloudflare has been flaky lately (#1303)" (#1306)
Revert " 👷 Add Codecov to CI, Smokeshow/Cloudflare has been flaky lately (#1303)" This reverts commit 759220d.
1 parent b8ded9b commit a1fdc57

File tree

2 files changed

+0
-11
lines changed

2 files changed

+0
-11
lines changed

‎.github/workflows/test.yml

-10
Original file line numberDiff line numberDiff line change
@@ -74,23 +74,13 @@ jobs:
7474
env:
7575
COVERAGE_FILE: coverage/.coverage.${{ runner.os }}-py${{ matrix.python-version }}-${{ matrix.pydantic-version }}
7676
CONTEXT: ${{ runner.os }}-py${{ matrix.python-version }}
77-
# TODO: if removing Smokeshow, and moving only to Codecov, remove this
78-
# Upload files before running Codecov, as it generates an extra file coverage/coverage.xml, and that breaks coverage-combine
7977
- name: Store coverage files
8078
uses: actions/upload-artifact@v4
8179
with:
8280
name: coverage-${{ matrix.python-version }}-${{ matrix.pydantic-version }}
8381
path: coverage
8482
include-hidden-files: true
85-
- uses: codecov/codecov-action@v5
86-
with:
87-
fail_ci_if_error: true
88-
files: ./coverage.xml
89-
name: codecov-umbrella
90-
token: ${{ secrets.CODECOV_TOKEN }}
91-
verbose: true
9283

93-
# TODO: if removing Smokeshow, and moving only to Codecov, remove this
9484
coverage-combine:
9585
needs:
9686
- test

‎scripts/test.sh

-1
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,3 @@ coverage run -m pytest tests
77
coverage combine
88
coverage report
99
coverage html
10-
coverage xml

0 commit comments

Comments
 (0)