Skip to content

[Coverage] Cleanup lcov version - #2641

Merged
soumeh01 merged 3 commits into
mainfrom
fix-cov
Oct 5, 2026
Merged

soumeh01 merged 3 commits into
mainfrom
fix-cov

Conversation

@soumeh01

@soumeh01 soumeh01 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

The change started with investigating a Codecov upload failure, which was ultimately caused by a Codecov service outage. During the investigation, I identified an opportunity to clean up and improve the workflow.

Changes

  • Use the GitHub runner's default version of lcov
  • Update the version of the Codecov GitHub Action
  • Remove deprecated option in command line

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Test Results

  2 files   14 suites   13m 36s ⏱️
482 tests 482 ✅ 0 💤 0 ❌
964 runs  964 ✅ 0 💤 0 ❌

Results for commit 64ca887.

♻️ This comment has been updated with latest results.

@soumeh01
soumeh01 force-pushed the fix-cov branch 6 times, most recently from b20ec5c to f332cfe Compare October 5, 2026 08:33
@soumeh01
soumeh01 marked this pull request as ready for review October 5, 2026 10:18
@codecov

codecov Bot commented Oct 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.00%. Comparing base (30f5e81) to head (64ca887).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2641      +/-   ##
==========================================
+ Coverage   79.00%   86.00%   +6.99%     
==========================================
  Files         207      207              
  Lines       33286    33286              
  Branches    19315        0   -19315     
==========================================
+ Hits        26298    28628    +2330     
+ Misses       4662     4658       -4     
+ Partials     2326        0    -2326     
Flag Coverage Δ
buildmgr-cov 89.51% <ø> (+9.80%) ⬆️
ctrace-cov 100.00% <ø> (ø)
packchk-cov 80.09% <ø> (+10.24%) ⬆️
packgen-cov 88.66% <ø> (+6.73%) ⬆️
projmgr-cov 95.02% <ø> (+7.05%) ⬆️
svdconv-cov 69.91% <ø> (+9.59%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 84 files with indirect coverage changes

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

@jkrech jkrech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - the fact that the coverage action passes gives me confidence that there is no random typo in the changes.

@soumeh01
soumeh01 merged commit b3b707f into main Oct 5, 2026
110 checks passed
@soumeh01
soumeh01 deleted the fix-cov branch October 5, 2026 10:32
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