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 codecov to this repo #923

Merged
merged 2 commits into from
Sep 13, 2023
Merged

Add codecov to this repo #923

merged 2 commits into from
Sep 13, 2023

Conversation

mjeffryes
Copy link
Member

@mjeffryes mjeffryes commented Aug 31, 2023

Modifies make test_provider to collect coverage profile files and adds a ci action to push them to codecov.

An example of the codecov analysis can be see in the comment below and at https://app.codecov.io/gh/pulumi/pulumi-google-native/commit/e2e535ad17dd7eee6a5509a3495a0a6dad2ca7b3

@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@mjeffryes mjeffryes added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Aug 31, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 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 ☂️

@github-actions
Copy link

github-actions bot commented Sep 9, 2023

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@mjeffryes
Copy link
Member Author

Codecov has apparently updated to their commenting bot to not try to make diffs when there's no previous history, but you can still see the data in their UI

@mjeffryes mjeffryes requested a review from a team September 9, 2023 00:22
@mjeffryes mjeffryes merged commit 21d2fbd into master Sep 13, 2023
14 checks passed
@mjeffryes mjeffryes deleted the mjeffryes/codecov branch September 13, 2023 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants