Skip to content

Update Codecov test-results upload to supported action path#2970

Open
justinchuby with Copilot wants to merge 7 commits into
mainfrom
copilot/update-codecov-pipeline-version
Open

Update Codecov test-results upload to supported action path#2970
justinchuby with Copilot wants to merge 7 commits into
mainfrom
copilot/update-codecov-pipeline-version

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

The CI run surfaced a deprecated Codecov test-results path (codecov/test-results-action@v1) and Node 20 deprecation warnings for that action. This PR updates the Codecov pipeline usage to the current supported invocation.

  • What changed

    • Replaced the dedicated test-results action:
      • codecov/test-results-action@v1
    • With codecov/codecov-action@v5 configured for test results upload.
  • Workflow impact

    • Keeps coverage upload step unchanged.
    • Moves test-results reporting onto the maintained codecov-action path (report_type: test_results) to align with Codecov’s deprecation guidance.
  • Patch excerpt

    - name: Upload test results to Codecov
      if: ${{ !cancelled() }}
      uses: codecov/codecov-action@v5
      with:
        token: ${{ secrets.CODECOV_TOKEN }}
        report_type: test_results

@justinchuby
justinchuby changed the base branch from copilot/follow-up-bincount-index-put to main July 23, 2026 14:51
@justinchuby

Copy link
Copy Markdown
Collaborator

@copilot cherrypick the fix and rebase from main

Copilot AI requested review from Copilot and removed request for Copilot July 23, 2026 14:55

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

@copilot cherrypick the fix and rebase from main

Done in 8ad6041a. I removed the unrelated changes so this PR now diffs only the Codecov fix against main.

Copilot AI requested a review from justinchuby July 23, 2026 14:57
@justinchuby
justinchuby enabled auto-merge (squash) July 23, 2026 14:57
@justinchuby
justinchuby requested a review from titaiwangms July 23, 2026 14:58
@codecov

codecov Bot commented Jul 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.64%. Comparing base (00de62e) to head (8ad6041).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2970   +/-   ##
=======================================
  Coverage   72.64%   72.64%           
=======================================
  Files         265      265           
  Lines       32192    32192           
  Branches     3038     3038           
=======================================
  Hits        23385    23385           
  Misses       7776     7776           
  Partials     1031     1031           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants