-
Notifications
You must be signed in to change notification settings - Fork 70
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
Added Code Coverage CI #450
Conversation
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Looks good! @henryleberre can review this PR since this is mostly code from him. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed a merge conflict and made some small tweaks. @sbryngelson you can squash merge when you're ready.
Will merge when CI passes |
Description
--gcov
as a build compile option to build with coverage files.Fixes #332
Type of change
Scope
How Has This Been Tested?
./mfc.sh build -j 8 --gcov
produced the coverage files without issue.build/staging/[id of a staging]/CMakeFiles/[staging].dir/fypp/
Test Configuration:
Ubuntu 22.04, Docker, and Github Actions
gcc-11, gcc-12, gcc-13
Checklist
docs/
)./mfc.sh format
before committing my code