-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 status badge #3845
Add codecov status badge #3845
Conversation
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
Signed-off-by: Suraj Singh <[email protected]>
5f5d965
to
61be6c5
Compare
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Suraj Singh <[email protected]>
Thanks @peterzhuamazon for pointing this out. Added a newline to segregate badge and logo. |
Since I setup each branch have its own coverage I think we can backport this. Example 1.x: https://app.codecov.io/gh/opensearch-project/opensearch/branch/1.x |
Gradle Check (Jenkins) Run Completed with:
|
Looks like there was already a PR out for adding readme badge. Let's not merge this one and close the other one. Converted to draft, will close once below PR is merged. |
That's a good point. We should have branch specific links. Though, I am thinking if there is a more generic fix which handles future branches as well. |
Dup of #1596? Can we close? |
Closed in favour of
Closed |
Description
Add readme code cov status badge
Issues Resolved
#1323
Related
#850
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.