Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Add coverage upload in test workflow and add badges in README #223

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

ftianli-amzn
Copy link

@ftianli-amzn ftianli-amzn commented Jul 10, 2020

Issue #, if available:

Description of changes:

  • Use Codecov to visualize the code coverage result
  • Add Coverage Report Upload action in test-workflow
  • Add several badges in README

This PR carries the code coverage visualization that already in index-management repo to alerting
opendistro-for-elasticsearch/index-management#230
opendistro-for-elasticsearch/index-management#231
opendistro-for-elasticsearch/index-management#232

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@codecov
Copy link

codecov bot commented Jul 10, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@2ae4247). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #223   +/-   ##
=========================================
  Coverage          ?   76.14%           
  Complexity        ?      157           
=========================================
  Files             ?       63           
  Lines             ?     2586           
  Branches          ?      402           
=========================================
  Hits              ?     1969           
  Misses            ?      421           
  Partials          ?      196           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ae4247...01602c2. Read the comment docs.

Copy link
Contributor

@dbbaughe dbbaughe left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Contributor

@bowenlan-amzn bowenlan-amzn left a comment

Choose a reason for hiding this comment

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

👏

@ftianli-amzn ftianli-amzn merged commit fed0a09 into opendistro-for-elasticsearch:master Jul 10, 2020
skkosuri-amzn pushed a commit that referenced this pull request Aug 13, 2020
 - Use Codecov to visualize the code coverage result
 - Add Coverage Report Upload action in test-workflow
 - Add several badges in README
tlfeng pushed a commit that referenced this pull request Feb 6, 2021
 - Use Codecov to visualize the code coverage result
 - Add Coverage Report Upload action in test-workflow
 - Add several badges in README
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants