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

chore: add upload_coverage_report.yml #697

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

vil02
Copy link
Member

@vil02 vil02 commented Nov 19, 2023

This repository exists on codecov. Since being able to easily browse through coverage reports it extremely useful, I decided to add a workflow uploading the report there. This PR is similar to TheAlgorithms/Rust#600.

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

@vil02 vil02 marked this pull request as ready for review November 19, 2023 14:58
Copy link

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

@github-actions github-actions bot added the stale label Dec 20, 2023
@vil02
Copy link
Member Author

vil02 commented Dec 20, 2023

@tjgurwara99, @raklaptudirm, @yanglbme: please dont-close.

@vil02 vil02 force-pushed the integrate_with_codecov branch from 12084aa to 710e353 Compare December 20, 2023 04:58
@vil02 vil02 force-pushed the integrate_with_codecov branch from 8f3d959 to 9d81f1d Compare January 13, 2024 18:53
Copy link
Member

@tjgurwara99 tjgurwara99 left a comment

Choose a reason for hiding this comment

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

I was confused by the ellipses at the end of the document but it seems to be in the Yaml spec.

@raklaptudirm raklaptudirm merged commit ea26922 into TheAlgorithms:master Jan 23, 2024
5 checks passed
@vil02 vil02 deleted the integrate_with_codecov branch January 23, 2024 08:09
@vil02 vil02 mentioned this pull request Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants