This repository has been archived by the owner on Aug 2, 2022. It is now read-only.
Get aggregate code coverage, add coverage upload in test workflow and add badges in README #88
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
alerting
repo: https://github.com/opendistro-for-elasticsearch/alerting/blob/master/build-tools/merged-coverage.gradleThis PR carries the code coverage visualization that already in
index-management
and ‘alerting’ repo tojob-scheduler
opendistro-for-elasticsearch/index-management#230
opendistro-for-elasticsearch/index-management#231
opendistro-for-elasticsearch/index-management#232
opendistro-for-elasticsearch/index-management#251
opendistro-for-elasticsearch/alerting#223
opendistro-for-elasticsearch/alerting#231
Concern: I think we should discuss which class is needed to do the code coverage analysis, and which don't.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.