From 16ca6734b5ee8aa28d027a85929430fec1f1d24d Mon Sep 17 00:00:00 2001 From: Darrel O'Pry Date: Mon, 3 Nov 2025 11:22:55 -0500 Subject: [PATCH] chore: defer codecov notify until all jobs done --- .github/workflows/test.yml | 20 ++++++++++++++++++++ codecov.yml | 8 ++++++++ 2 files changed, 28 insertions(+) create mode 100644 codecov.yml diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 577cb56a5..8ba81afc7 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -117,10 +117,30 @@ jobs: run: npm run build working-directory: tests/app/rp + codecov-notify: + needs: + - test-package + - test-demo-rp + runs-on: ubuntu-latest + name: Codecov Notify + steps: + # - tell codecov to send notifications now that all jobs are complete. + # without this, codecov may notify before all coverage reports have been uploaded. + # `codecov: notify: manual_trigger: true` must be set in codecov.yml, to prevent + # processing on every upload. + # - preferred to after_n_builds so we don't need to update that number every + # time we add/remove jobs. + - name: Notify Codecov + uses: codecov/codecov-action@v5 + with: + run_command: 'send-notifications' + use_oidc: true + success: needs: - test-package - test-demo-rp + - codecov-notify runs-on: ubuntu-latest name: Test successful steps: diff --git a/codecov.yml b/codecov.yml new file mode 100644 index 000000000..6dd938982 --- /dev/null +++ b/codecov.yml @@ -0,0 +1,8 @@ + +codecov: + # since we're uploading coverage from multiple parallel jobs,Codecov doesn't + # inherently know when all reports have been uploaded for a given commit. To + # prevent premature processing and ensure a complete report, we send + # notification when all jobs are done. + notify: + manual_trigger: true \ No newline at end of file