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

[repo] Code Coverage badges #1838

Merged
merged 2 commits into from
May 23, 2024
Merged

Conversation

Kielek
Copy link
Contributor

@Kielek Kielek commented May 23, 2024

Fixes #116.

Changes

Code Coverage badges for each package.
Part of them is common (AspNet, PersistentStorage).

Separate codecoverage enabled by #1833
Some of the badges display unknown - flags was not yet cached by the codecov.io.

Example for AspNet:

codecov.io

For significant contributions please make sure you have completed the following items:

  • [ ] Appropriate CHANGELOG.md updated for non-trivial changes
  • [ ] Design discussion issue #
  • [ ] Changes in public API reviewed

@Kielek Kielek added documentation Improvements or additions to documentation infra Infra work - CI/CD, code coverage, linters labels May 23, 2024
@Kielek Kielek requested a review from a team May 23, 2024 07:40
Copy link
Member

@MikeGoldsmith MikeGoldsmith left a comment

Choose a reason for hiding this comment

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

Look great, thanks @Kielek 🚀

Copy link
Member

@cijothomas cijothomas left a comment

Choose a reason for hiding this comment

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

💯 love the consistent readme files for all components!

@cijothomas cijothomas merged commit f1fd71f into open-telemetry:main May 23, 2024
48 checks passed
@Kielek Kielek deleted the codecov-shields branch May 23, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation infra Infra work - CI/CD, code coverage, linters
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Badge for Code Coverage