chore: Reduce codecov pushes

pushes to non-develop branches will not send coverage reports. This might reduce the coverage variability issue we're having.
mermaid-js · Jul 6, 2023 · 4d0c461 · 4d0c461
1 parent db30f21
commit 4d0c461
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml
@@ -47,7 +47,8 @@ jobs:
           VITEST_COVERAGE: true
       - name: Upload Coverage to Codecov
         uses: codecov/codecov-action@v3
-        if: steps.cypress.conclusion == 'success'
+        # Run step only pushes to develop and pull_requests
+        if: ${{ steps.cypress.conclusion == 'success' && (github.event_name == 'pull_request' || github.ref == 'refs/heads/develop')}}
         with:
           files: coverage/cypress/lcov.info
           flags: e2e

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -43,6 +43,8 @@ jobs:
 
       - name: Upload Coverage to Codecov
         uses: codecov/codecov-action@v3
+        # Run step only pushes to develop and pull_requests
+        if: ${{ github.event_name == 'pull_request' || github.ref == 'refs/heads/develop' }}
         with:
           files: ./coverage/vitest/lcov.info
           flags: unit