Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add code coverage GitHub Action #225

Merged
merged 4 commits into from
Aug 2, 2022
Merged

Add code coverage GitHub Action #225

merged 4 commits into from
Aug 2, 2022

Conversation

aloisklink
Copy link
Contributor

It seems like the CMake/CodeCoverage.cmake file wasn't being used, so I quickly added a GitHub Action for it.

Currently, the coverage action runs after all tests pass, since it only makes sense to check coverage if tests pass. I've also kept it as a separate cmake action, as it requires lcov, which is not normally installed by people.

@github-actions
Copy link

github-actions bot commented Aug 2, 2022

LCOV of commit d717307 during Build #853

Summary coverage rate:
  lines......: 43.7% (5762 of 13186 lines)
  functions..: 59.7% (549 of 919 functions)
  branches...: no data found

Files changed coverage rate: n/a

Copy link
Contributor

@mereacre mereacre left a comment

Choose a reason for hiding this comment

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

Great ideas.

Co-authored-by: Alois Klink <alois@nquiringminds.com>
@mereacre mereacre merged commit 31eab68 into main Aug 2, 2022
@mereacre mereacre deleted the code-coverage branch August 2, 2022 15:05
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.

None yet

2 participants