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

Coverage reporting and upload to codecov. #14

Merged
merged 3 commits into from
Oct 18, 2023
Merged

Conversation

samcunliffe
Copy link
Member

@samcunliffe samcunliffe commented Oct 17, 2023

Adds coverage reporting and upload of the report to codecov.io.
We'll get @codecov bot to comment on PRs and ensure we don't lose coverage (in the same way as we do for dxss).

Blocked by

The tests (and therefore coverage upload) will fail until

... is merged.

@samcunliffe
Copy link
Member Author

[ Sorry for the noise and force pushing... now all good. ]

Copy link
Collaborator

@matt-graham matt-graham left a comment

Choose a reason for hiding this comment

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

Everything looks good from what I can see from checking against codecov / codecov/codecov-action docs.

@codecov
Copy link

codecov bot commented Oct 18, 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 ☂️

@samcunliffe samcunliffe merged commit e48a665 into main Oct 18, 2023
4 checks passed
@samcunliffe samcunliffe deleted the sc/coverage-to-codecov branch October 18, 2023 17:53
@samcunliffe
Copy link
Member Author

codecov 💪

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.

2 participants