⚠ 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

Summary by CodeRabbit

  • Chores
    • Improved test coverage reporting in CI workflow configuration.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 26, 2026

📝 Walkthrough

Walkthrough

The CI workflow's coverage generation step was modified to explicitly specify the module's include and src directories in LCOV, replacing a generic glob pattern. The command was reformatted across multiple lines while preserving the gcov tool selection and output path behavior.

Changes

Cohort / File(s) Summary
CI Workflow Coverage Configuration
.github/workflows/ci.yml
Modified LCOV coverage generation step to add explicit includes for module include/ and src/ directories; reformatted command across multiple lines for readability

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A workflow adjusted with care,
LCOV paths now laid bare,
Include and src directories bright,
Coverage data formatted just right,
CI pipelines skip and leap,
With coverage scope now deep! ✨

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: modifying the coverage generation to include only files from the corosio module's include/ and src/ directories.

✏️ 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://83.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 16:18:18 UTC

@sgerbino sgerbino merged commit f78a90f into cppalliance:develop Jan 26, 2026
13 of 14 checks passed
@sgerbino sgerbino deleted the fix/coverage branch January 26, 2026 16:49
@codecov
Copy link

codecov bot commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 37.83%. Comparing base (1ec549a) to head (6736247).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #83      +/-   ##
===========================================
- Coverage    38.03%   37.83%   -0.20%     
===========================================
  Files          258      258              
  Lines        14179    14161      -18     
  Branches      1442     1442              
===========================================
- Hits          5393     5358      -35     
  Misses        8738     8738              
- Partials        48       65      +17     

see 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ec549a...6736247. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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