Separate codecov steps for PRs and non-PRs

Author: shaypal5

.github/workflows/ci-test.yml

@@ -144,15 +144,26 @@ jobs:
         if: matrix.backend == 'redis'
         run: pytest -m redis --cov=cachier --cov-report=term --cov-report=xml:cov.xml
 
-      - name: Upload coverage to Codecov
+      - name: Upload coverage to Codecov (non-PRs)
+        if: github.event_name != 'pull_request'   # pushes, scheduled, etc.
         continue-on-error: true
         uses: codecov/codecov-action@v5
         with:
           fail_ci_if_error: true
-          name: codecov-umbrella
           token: ${{ secrets.CODECOV_TOKEN }} # required
           flags: ${{ matrix.backend }}
 
+      # We need a separate step for PRs because PRs do not have access to
+      # secrets; hence, provide the token will give it an empty value,
+      # resulting in an authentication error.
+      - name: Upload coverage to Codecov (PRs)
+        if: github.event_name == 'pull_request' # PRs only
+        continue-on-error: true
+        uses: codecov/codecov-action@v5
+        with:
+          fail_ci_if_error: true
+          flags: ${{ matrix.backend }}
+
   testing-guardian:
     runs-on: ubuntu-latest
     needs: pytester