Skip to content

Use gcovr instead of lcov in GitHub CI job#27

Merged
msteinert merged 1 commit into
mainfrom
gcovr-workflow
Aug 20, 2025
Merged

Use gcovr instead of lcov in GitHub CI job#27
msteinert merged 1 commit into
mainfrom
gcovr-workflow

Conversation

@rdmark

@rdmark rdmark commented Aug 19, 2025

Copy link
Copy Markdown
Collaborator

The gcovr tool has more solid support for filtering files to analyze for code coverage

@rdmark rdmark force-pushed the gcovr-workflow branch 2 times, most recently from 6bff1bd to 69e6069 Compare August 19, 2025 19:53
@github-actions

github-actions Bot commented Aug 19, 2025

Copy link
Copy Markdown

File Coverage Lines Branches
All files 68% 73% 63%
bstring/bstraux.c 55% 64% 46%
bstring/bstrlib.c 73% 76% 69%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against cff61bc

The gcovr tool has more solid support for filtering files to analyze for code coverage
@sonarqubecloud

Copy link
Copy Markdown

@rdmark rdmark marked this pull request as ready for review August 20, 2025 05:15
@rdmark rdmark requested a review from msteinert August 20, 2025 05:15
@rdmark

rdmark commented Aug 20, 2025

Copy link
Copy Markdown
Collaborator Author

@msteinert How about this for a solution? I dropped lcov and use gcovr solely. The latter has better support for per-project configurations, so it integrates better with meson and CI.

@msteinert

Copy link
Copy Markdown
Owner

@msteinert How about this for a solution? I dropped lcov and use gcovr solely. The latter has better support for per-project configurations, so it integrates better with meson and CI.

🔥 🔥 🔥

@msteinert msteinert merged commit ed98994 into main Aug 20, 2025
16 checks passed
@rdmark rdmark deleted the gcovr-workflow branch August 20, 2025 17:56
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