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

add code coverage (via upstream branch) #2

Merged
merged 2 commits into from
Jan 23, 2024
Merged

add code coverage (via upstream branch) #2

merged 2 commits into from
Jan 23, 2024

Conversation

SimeonEhrig
Copy link
Member

Duplicate of PR #1.

Use branch in upstream repository instead fork because it looks like the upload to codecov.io cannot be tested with a fork.

@codecov-commenter
Copy link

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

@SimeonEhrig SimeonEhrig marked this pull request as ready for review January 23, 2024 12:58
@SimeonEhrig
Copy link
Member Author

The CI passed, I can check the code coverage on Codecov and the bot answered. Only displaying the code coverage in the PR is missing but the bot mentioned, that will happen after I merge the PR. Therefore I merge it now.

@SimeonEhrig SimeonEhrig merged commit 3207d45 into main Jan 23, 2024
2 checks passed
@SimeonEhrig SimeonEhrig deleted the codeCoverage branch January 23, 2024 13:00
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