From ac4fd2a1115e57212cf1fd125fceab8323ba9310 Mon Sep 17 00:00:00 2001 From: Matt Dawkins Date: Tue, 27 Jan 2026 11:05:16 -0500 Subject: [PATCH] fix: correct CI pre-commit invocation and coverage upload path Two fixes in the testing workflow: 1. Remove bare `pre-commit` call (no subcommand) which just prints help text and wastes CI time. Only `pre-commit run --all-files` is needed. 2. Fix Codecov upload path from `./coverage/coverage.xml` to `./coverage.xml`. pytest-cov with `--cov-report=xml` writes coverage.xml to the working directory by default, not a coverage/ subdirectory. Co-Authored-By: Claude Opus 4.5 --- .github/workflows/testing.yaml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/.github/workflows/testing.yaml b/.github/workflows/testing.yaml index a524773..7704566 100644 --- a/.github/workflows/testing.yaml +++ b/.github/workflows/testing.yaml @@ -39,7 +39,6 @@ jobs: - name: Check with pre-commit run: | - SKIP=hadolint pre-commit SKIP=hadolint pre-commit run --all-files - name: Run tests and coverage @@ -52,6 +51,6 @@ jobs: uses: codecov/codecov-action@v5 with: token: ${{ secrets.BATBOT_CODECOV_TOKEN }} - files: ./coverage/coverage.xml + files: ./coverage.xml env_vars: OS,PYTHON fail_ci_if_error: true