From a1fdc57271da1993a79587ecde12a5cb5b2255f3 Mon Sep 17 00:00:00 2001 From: Sofie Van Landeghem Date: Fri, 21 Feb 2025 15:45:27 +0100 Subject: [PATCH] =?utf8?q?=F0=9F=91=B7=20Revert=20"Add=20Codecov=20to=20CI?= =?utf8?q?,=20Smokeshow/Cloudflare=20has=20been=20flaky=20lately=20(#1303)?= =?utf8?q?"=20(#1306)?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Revert " 👷 Add Codecov to CI, Smokeshow/Cloudflare has been flaky lately (#1303)" This reverts commit 759220d5928af4a01c205a039eb9826456a475cd. --- .github/workflows/test.yml | 10 ---------- scripts/test.sh | 1 - 2 files changed, 11 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index c924e54e..9126db93 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -74,23 +74,13 @@ 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 diff --git a/scripts/test.sh b/scripts/test.sh index 826ae903..ff4b114b 100755 --- a/scripts/test.sh +++ b/scripts/test.sh @@ -7,4 +7,3 @@ coverage run -m pytest tests coverage combine coverage report coverage html -coverage xml -- 2.47.2