add a coverage drop threshold to limit codecov failure spam #803

Merged
merged 2 commits into from Feb 21, 2017

Conversation

Projects
None yet
3 participants
Contributor

lbergelson commented Feb 10, 2017

Codecov is adding failure messages for very small coverage changes that aren't worrisome. I'm adding a threshold that should hopefully fix it so only more significant coverage drops are reported as failures.

lbergelson added some commits Feb 10, 2017

@lbergelson lbergelson add a coverage drop threshold to limit codecov failure spam
9beb81d
@lbergelson lbergelson making codcov.yml a hidden file
70b202d

codecov-io commented Feb 10, 2017 edited

Codecov Report

Merging #803 into master will increase coverage by 0.01%.

@@             Coverage Diff              @@
##             master     #803      +/-   ##
============================================
+ Coverage     64.73%   64.74%   +0.01%     
- Complexity     7134     7138       +4     
============================================
  Files           525      525              
  Lines         31679    31679              
  Branches       5414     5414              
============================================
+ Hits          20506    20512       +6     
+ Misses         9028     9026       -2     
+ Partials       2145     2141       -4
Impacted Files Coverage Δ Complexity Δ
...main/java/htsjdk/samtools/SAMRecordSetBuilder.java 89.11% <ø> (+0.8%) 64% <ø> (+2%)
...a/htsjdk/samtools/cram/encoding/rans/Decoding.java 85.71% <ø> (+3.57%) 12% <ø> (+1%)
...samtools/util/AsyncBlockCompressedInputStream.java 76% <ø> (+4%) 13% <ø> (+1%)

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 18508d4...70b202d. Read the comment docs.

Contributor

droazen commented Feb 21, 2017

👍 merge when ready

lbergelson was assigned by droazen Feb 21, 2017

@lbergelson lbergelson merged commit cabe78f into master Feb 21, 2017

4 checks passed

codecov/patch Coverage not affected when comparing 18508d4...70b202d
Details
codecov/project 64.74% (+0.01%) compared to 18508d4
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

lbergelson deleted the lb_add_codecov_threshold branch Feb 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment