Skip to content

Conversation

NWilson
Copy link
Member

@NWilson NWilson commented Sep 17, 2025

  • Add maint/RunCoverage script to allow gathering Clang coverage locally more easily
  • Add LCOV exclusion comments to unreachable code
  • Add a filtering script so that the CI job respects LCOV_EXCL_* comments

@zherczeg
Copy link
Collaborator

I thought this patch adds more test cases :)

@NWilson
Copy link
Member Author

NWilson commented Sep 18, 2025

It does add a very few extra lines of coverage - up by 0.5% apparently.

But there are definitely more PRs coming.

In the next PR we'll get to 100% function coverage - we still have some totally unhit functions, which aren't especially important but it would still be better to avoid that.

@NWilson NWilson merged commit 379123d into master Sep 20, 2025
35 checks passed
@NWilson NWilson deleted the user/niwilson/lcov-improvements branch September 20, 2025 14:10
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