Skip to content

chore: Update the codecov action.#280

Merged
Artmann merged 3 commits intomainfrom
chris/fix-codecov
Jan 8, 2026
Merged

chore: Update the codecov action.#280
Artmann merged 3 commits intomainfrom
chris/fix-codecov

Conversation

@Artmann
Copy link
Copy Markdown
Contributor

@Artmann Artmann commented Jan 8, 2026

We are getting this warning in the CI

This action is being deprecated in favor of 'codecov-action'.
Please update CI accordingly to use 'codecov-action@v5' with
'report_type: test_results'.
The 'codecov-action' should and can be run at least once for
coverage and once for test results

Summary by CodeRabbit

  • Chores
    • Updated CI/CD test result reporting infrastructure to use the latest version of the testing framework integration.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 8, 2026

📝 Walkthrough

Walkthrough

The CI workflow updates the Codecov integration for test result uploads. The legacy test-results-action (v1) is replaced with codecov-action (v5). An explicit report_type: test_results parameter is added to the newer action, clarifying the upload intent. The configuration maintains existing parameters for OIDC authentication, file paths, and failure handling. This is a direct migration to Codecov's current tooling with no functional behavior changes to the CI pipeline.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed Title accurately describes the main change: updating the Codecov GitHub action from v1 to v5.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0%. Comparing base (ada6fe3) to head (ccc231d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@     Coverage Diff     @@
##   main   #280   +/-   ##
===========================
===========================
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Artmann Artmann marked this pull request as ready for review January 8, 2026 10:30
@Artmann Artmann requested a review from a team as a code owner January 8, 2026 10:30
coderabbitai[bot]
coderabbitai Bot previously approved these changes Jan 8, 2026
@Artmann Artmann requested review from saltenasl and tkislan January 8, 2026 11:01
tkislan
tkislan previously approved these changes Jan 8, 2026
Configure mocha to use multi-reporters so test-results.xml is generated
during test runs. This fixes the "file not found" error in CI when
uploading test results to Codecov.
@Artmann Artmann dismissed stale reviews from tkislan and coderabbitai[bot] via ccc231d January 8, 2026 11:07
@Artmann Artmann merged commit 1252f61 into main Jan 8, 2026
12 checks passed
@Artmann Artmann deleted the chris/fix-codecov branch January 8, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants