Adding OpenJDK8 to travis configuration #875

Merged
merged 1 commit into from May 17, 2017

Conversation

Projects
None yet
3 participants
Contributor

jacarey commented May 16, 2017

Added OpenJDK8 as a build configuration to be tested via Travis.

Checklist

  • Code compiles correctly
  • New tests covering changes and new functionality
  • All tests passing
  • Extended the README / documentation, if necessary
  • Is not backward compatible (breaks binary or source compatibility)
@jacarey jacarey Adding OpenJDK8 to travis configuration
e2801ee

codecov-io commented May 16, 2017 edited

Codecov Report

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

@@               Coverage Diff               @@
##              master      #875       +/-   ##
===============================================
+ Coverage     65.003%   65.025%   +0.022%     
- Complexity      7231      7233        +2     
===============================================
  Files            528       528               
  Lines          31894     31894               
  Branches        5444      5444               
===============================================
+ Hits           20732     20739        +7     
+ Misses          9016      9011        -5     
+ Partials        2146      2144        -2
Impacted Files Coverage Δ Complexity Δ
...dk/samtools/seekablestream/SeekableHTTPStream.java 54.545% <0%> (-1.515%) 9% <0%> (-1%)
...va/htsjdk/samtools/util/AsyncBufferedIterator.java 69.565% <0%> (+1.087%) 17% <0%> (ø) ⬇️
...a/htsjdk/samtools/cram/encoding/rans/Decoding.java 85.714% <0%> (+3.571%) 12% <0%> (+1%) ⬆️
...samtools/util/AsyncBlockCompressedInputStream.java 76% <0%> (+4%) 13% <0%> (+1%) ⬆️
...sjdk/samtools/util/Md5CalculatingOutputStream.java 79.487% <0%> (+7.692%) 9% <0%> (+1%) ⬆️
@pshapiro4broad

pshapiro4broad approved these changes May 17, 2017 edited

👍

Does it seem odd to you that this change affected coverage?

@jacarey jacarey merged commit 1b27398 into master May 17, 2017

4 of 5 checks passed

codecov/changes 1 file has unexpected coverage changes not visible in diff.
Details
codecov/patch Coverage not affected when comparing e1cf07e...e2801ee
Details
codecov/project 65.025% (+0.022%) compared to e1cf07e
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

jacarey deleted the jc_DSDEGP-1326 branch May 17, 2017

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