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

Add Codecov configuration to set a coverage threshold to pass the check on a commit #231

Merged
merged 2 commits into from
Aug 10, 2020

Conversation

ftianli-amzn
Copy link

Issue #, if available:

Description of changes:

Add codecov.yml configuration file and set the minimum target coverage to 70% while allowing a 1% drop.
Without those config, the check on a commit will not pass.
Reference: opendistro-for-elasticsearch/index-management#251

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 28, 2020

Codecov Report

Merging #231 into master will increase coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #231      +/-   ##
============================================
+ Coverage     76.14%   76.25%   +0.11%     
  Complexity      157      157              
============================================
  Files            63       63              
  Lines          2586     2586              
  Branches        402      402              
============================================
+ Hits           1969     1972       +3     
+ Misses          421      418       -3     
  Partials        196      196              
Impacted Files Coverage Δ Complexity Δ
...distroforelasticsearch/alerting/core/JobSweeper.kt 71.89% <0.00%> (+0.54%) 35.00% <0.00%> (ø%)
...roforelasticsearch/alerting/alerts/AlertIndices.kt 67.46% <0.00%> (+1.58%) 0.00% <0.00%> (ø%)

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 fa7dec4...7f925c6. Read the comment docs.

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