From 978803094bb8a739c7b0f85181144f393472b6dd Mon Sep 17 00:00:00 2001 From: Francesco Gringl-Novy Date: Mon, 13 Apr 2026 14:06:41 +0200 Subject: [PATCH 1/2] chore(ci): Skip coverage check for playwright browser tests --- .github/workflows/build.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 1bf07adf661c..56668f0e91f0 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -605,6 +605,7 @@ jobs: with: token: ${{ secrets.GITHUB_TOKEN }} directory: dev-packages/browser-integration-tests + enable-coverage: false name: browser-playwright-${{ matrix.bundle }}-${{ matrix.project }}${{ matrix.shard && format('-{0}', matrix.shard) || '' }} @@ -669,6 +670,7 @@ jobs: with: token: ${{ secrets.GITHUB_TOKEN }} directory: dev-packages/browser-integration-tests + enable-coverage: false name: browser-loader-${{ matrix.bundle }} job_check_for_faulty_dts: From 472526a34524a5bf0e0de450828b3f9d8a59e126 Mon Sep 17 00:00:00 2001 From: Francesco Gringl-Novy Date: Mon, 13 Apr 2026 14:07:03 +0200 Subject: [PATCH 2/2] fix(ci): Remove codecov steps from jobs that produce no coverage/JUnit data MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Several CI jobs were running `getsentry/codecov-action` despite not producing any coverage or JUnit XML files, causing noisy warnings in every CI run: - Node integration tests: coverage disabled in vitest config, custom reporters without JUnit output - Node-core integration tests: same as above - E2E tests: use ts-node/Playwright, not vitest — no coverage or JUnit - Remix tests: custom vitest config without coverage or JUnit reporter Also disables coverage upload for Playwright browser integration tests (both regular and loader), which only produce JUnit XML but not coverage files. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/build.yml | 37 ------------------------------------- 1 file changed, 37 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 56668f0e91f0..38e12b1fa422 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -739,15 +739,6 @@ jobs: working-directory: dev-packages/node-integration-tests run: yarn test - - name: Parse and Upload Coverage - if: cancelled() == false - continue-on-error: true - uses: getsentry/codecov-action@main - with: - token: ${{ secrets.GITHUB_TOKEN }} - directory: dev-packages/node-integration-tests - name: node-integration-${{ matrix.node }}${{ matrix.typescript && format('-ts{0}', matrix.typescript) || '' }} - job_node_core_integration_tests: name: Node (${{ matrix.node }})${{ (matrix.typescript && format(' (TS {0})', matrix.typescript)) || '' }} Node-Core @@ -789,16 +780,6 @@ jobs: working-directory: dev-packages/node-core-integration-tests run: yarn test - - name: Parse and Upload Coverage - if: cancelled() == false - continue-on-error: true - uses: getsentry/codecov-action@main - with: - token: ${{ secrets.GITHUB_TOKEN }} - directory: dev-packages/node-core-integration-tests - name: - node-core-integration-${{ matrix.node }}${{ matrix.typescript && format('-ts{0}', matrix.typescript) || ''}} - job_cloudflare_integration_tests: name: Cloudflare Integration Tests needs: [job_get_metadata, job_build] @@ -858,15 +839,6 @@ jobs: cd packages/remix yarn test:integration:ci - - name: Parse and Upload Coverage - if: cancelled() == false - continue-on-error: true - uses: getsentry/codecov-action@main - with: - directory: packages/remix - token: ${{ secrets.GITHUB_TOKEN }} - name: ${{ matrix.node }} - job_e2e_prepare: name: Prepare E2E tests # We want to run this if: @@ -1055,15 +1027,6 @@ jobs: retention-days: 7 if-no-files-found: ignore - - name: Parse and Upload Coverage - if: cancelled() == false - continue-on-error: true - uses: getsentry/codecov-action@main - with: - directory: dev-packages/e2e-tests - token: ${{ secrets.GITHUB_TOKEN }} - name: e2e-${{ matrix.test-application }} - # - We skip optional tests on release branches job_optional_e2e_tests: name: E2E ${{ matrix.label || matrix.test-application }} Test (optional)