⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@sgerbino
Copy link
Collaborator

@sgerbino sgerbino commented Jan 26, 2026

Use CodeCov actions to generate coverage report for branches like develop. Don't fail CI based on coverage.

Summary by CodeRabbit

Chores

  • Updated coverage status checks to be informational—coverage reports no longer fail CI workflows
  • Migrated coverage reporting to an improved GitHub Action-based integration for better reliability and maintainability

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 26, 2026

📝 Walkthrough

Walkthrough

The pull request updates the Codecov integration across two configuration files. A new coverage status block is added to .codecov.yml to make coverage checks informational, while .github/workflows/ci.yml replaces the curl-based Codecov integration with the official codecov/codecov-action@v5 GitHub Action, moving token handling to action inputs.

Changes

Cohort / File(s) Summary
Codecov Configuration
.codecov.yml
Adds new coverage status configuration to make both project and patch coverage checks informational (report-only mode, never fails CI)
CI Workflow Integration
.github/workflows/ci.yml
Replaces curl-based Codecov upload with codecov/codecov-action@v5; renames "Codecov" step to "Generate Coverage Report"; removes env-injected CODECOV_TOKEN and moves token handling to action inputs; adds separate "Coverage Summary" step for GitHub output

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • codecov token #71: Modifies the same Codecov integration in .github/workflows/ci.yml with alternative approaches to CODECOV_TOKEN handling in the GitHub Action workflow.

Poem

🐰 Hops of joy for coverage flows,
From curl to action, progress grows,
Informational now, never red,
CI runs smoothly ahead!

🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'CodeCov improvements' is vague and generic, using non-descriptive language that doesn't clearly convey the specific changes made in the PR. Consider a more specific title like 'Use CodeCov GitHub Action and make coverage checks informational' to better reflect the actual changes.
✅ Passed checks (1 passed)
Check name Status Explanation
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.

@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://82.corosio.prtest3.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-01-26 15:28:13 UTC

@codecov
Copy link

codecov bot commented Jan 26, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@sgerbino sgerbino merged commit 1ec549a into cppalliance:develop Jan 26, 2026
16 checks passed
@sgerbino sgerbino deleted the fix/codecov branch January 26, 2026 15:50
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