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

ci: compute code coverage in the CI #57

Merged

Conversation

p9f
Copy link
Collaborator

@p9f p9f commented Oct 10, 2023

Codecov integration.

I disabled the comment as I think the ability to see the status check is enough - but happy to follow whatever other prefers.
Once it's merge to main we can add the codecov badge in the readme.

@codecov
Copy link

codecov bot commented Oct 10, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@p9f p9f force-pushed the ci-compute-code-coverage-in-the-ci branch from 4b40398 to 4cb979c Compare October 10, 2023 08:00
@p9f p9f marked this pull request as ready for review October 10, 2023 08:03
Copy link
Owner

@gustavoguichard gustavoguichard left a comment

Choose a reason for hiding this comment

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

Awesome ❤️

@gustavoguichard gustavoguichard merged commit cf85f2e into gustavoguichard:main Oct 10, 2023
4 checks passed
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