Skip to content

Added dockerfile and action trigger to run Code Coverage#89

Merged
montythind merged 6 commits intomainfrom
codeCoverage_88
Apr 24, 2025
Merged

Added dockerfile and action trigger to run Code Coverage#89
montythind merged 6 commits intomainfrom
codeCoverage_88

Conversation

@montythind
Copy link
Copy Markdown
Contributor

This pr get the code coverage for the project

closes #88

@codecov-commenter
Copy link
Copy Markdown

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 ☂️

Comment thread docker/Dockerfile.coverage Outdated
Comment thread docker/Dockerfile.coverage Outdated
@montythind montythind requested a review from K20shores April 24, 2025 15:48
Comment thread CMakeLists.txt Outdated
Copy link
Copy Markdown
Collaborator

@mattldawson mattldawson left a comment

Choose a reason for hiding this comment

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

Looks good! The only thing beyond what Kyle mentioned might be adding the codecov badge to the README

Comment thread cmake/CodeCoverage.cmake Outdated
@boulderdaze
Copy link
Copy Markdown
Contributor

Could you also edit the title of this PR, including what this PR addresses.

@montythind montythind changed the title created action trigger Added dockerfile and action trigger to run Code Coverage Apr 24, 2025
@montythind montythind requested a review from boulderdaze April 24, 2025 20:17
Copy link
Copy Markdown
Contributor

@boulderdaze boulderdaze left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the comments!

@montythind montythind merged commit ec956c2 into main Apr 24, 2025
35 checks passed
@montythind montythind deleted the codeCoverage_88 branch April 24, 2025 21:18
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.

Add Code Coverage and Docker

5 participants