Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

Ensure coverage for function line in LCOV report#1212

Merged
Spacetown merged 15 commits intogcovr:mainfrom
Spacetown:lcov_ensure_coverage_for_function_line
Jan 8, 2026
Merged

Ensure coverage for function line in LCOV report#1212
Spacetown merged 15 commits intogcovr:mainfrom
Spacetown:lcov_ensure_coverage_for_function_line

Conversation

@Spacetown
Copy link
Member

@Spacetown Spacetown commented Dec 2, 2025

Temporary merge the data for the LCOV report and ensure that there is always a line data for a function.

Related #1190

@codacy-production
Copy link

codacy-production bot commented Dec 2, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.06% (target: -1.00%) 100.00% (target: 97.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (ab1f628) 5357 5187 96.83%
Head commit (5e63f20) 5372 (+15) 5205 (+18) 96.89% (+0.06%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1212) 50 50 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@Spacetown
Copy link
Member Author

I'll decided to split this PR into smaller junks:

- Clarification if the option is excluding or removing the data.
- First do the removal processing and afterwards the exclusion. Old
  behavior was e.g. first exclude branches of a compiler generated
  function which was removed later.
Copy link
Contributor

@dlegaultbbry dlegaultbbry left a comment

Choose a reason for hiding this comment

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

LGTM

@Spacetown Spacetown marked this pull request as ready for review December 10, 2025 21:45
@Spacetown
Copy link
Member Author

@abrown-RV, @dlegaultbbry I think this can be merged because the open point seems to be related to the command call (#1190 (comment)).

@Spacetown Spacetown merged commit 36fce2a into gcovr:main Jan 8, 2026
46 checks passed
@Spacetown Spacetown deleted the lcov_ensure_coverage_for_function_line branch January 8, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants