Skip to content

ci: add Codecov integration to workflows#1143

Merged
Pouyanpi merged 3 commits intodevelopfrom
ci/codecoverage
May 2, 2025
Merged

ci: add Codecov integration to workflows#1143
Pouyanpi merged 3 commits intodevelopfrom
ci/codecoverage

Conversation

@Pouyanpi
Copy link
Collaborator

@Pouyanpi Pouyanpi commented Apr 24, 2025

The workflow is triggered on pushes and pull requests to the develop branch. Codecov integration ensures better visibility into test coverage metrics.

https://docs.codecov.com/docs/common-recipe-list

@Pouyanpi Pouyanpi changed the title ci: add Codecov integration to GitHub Actions ci: add Codecov integration to workflows Apr 24, 2025
@codecov-commenter
Copy link

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

The workflow is triggered on pushes and pull requests to the `develop`
branch. Codecov integration ensures better visibility into test
coverage metrics.
@Pouyanpi Pouyanpi requested a review from tgasser-nv May 1, 2025 11:26
@Pouyanpi Pouyanpi marked this pull request as ready for review May 1, 2025 11:26
@Pouyanpi Pouyanpi requested a review from cparisien May 1, 2025 14:13
Copy link
Collaborator

@tgasser-nv tgasser-nv left a comment

Choose a reason for hiding this comment

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

Love this idea of checking coverage automatically (especially patch coverage for new PRs). I guess we can't really test this prior to activating it in Github Actions .. let's get this PR merged and then we can do followup PRs if there are any issues. Thanks!

@Pouyanpi Pouyanpi merged commit 67b8b7a into develop May 2, 2025
11 checks passed
@Pouyanpi Pouyanpi deleted the ci/codecoverage branch May 2, 2025 06:28
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.

3 participants