Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the README with working test examples #1225

Merged
merged 1 commit into from
Nov 13, 2018

Conversation

lbergelson
Copy link
Member

@lbergelson lbergelson commented Nov 9, 2018

Description

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)

* The existing examples of how to run single tests are broken, updating
them with working examples.
* Resolves #984
@codecov-io
Copy link

codecov-io commented Nov 9, 2018

Codecov Report

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

@@               Coverage Diff               @@
##              master     #1225       +/-   ##
===============================================
+ Coverage     69.016%   69.031%   +0.015%     
- Complexity      8066      8069        +3     
===============================================
  Files            539       539               
  Lines          32552     32552               
  Branches        5488      5488               
===============================================
+ Hits           22466     22471        +5     
+ Misses          7890      7888        -2     
+ Partials        2196      2193        -3
Impacted Files Coverage Δ Complexity Δ
...dk/samtools/seekablestream/SeekableHTTPStream.java 58.824% <0%> (+1.471%) 14% <0%> (+1%) ⬆️
...va/htsjdk/samtools/sra/SRAIndexedSequenceFile.java 64.865% <0%> (+2.703%) 7% <0%> (+1%) ⬆️
...samtools/util/AsyncBlockCompressedInputStream.java 76% <0%> (+4%) 13% <0%> (+1%) ⬆️

@lbergelson
Copy link
Member Author

@pshapiro4broad Thank you.

@lbergelson lbergelson merged commit e0895a1 into master Nov 13, 2018
@lbergelson lbergelson deleted the lb_update_test_example_readme branch November 13, 2018 14:32
@jmthibault79 jmthibault79 requested review from jmthibault79 and removed request for jmthibault79 November 14, 2018 15:51
@jmthibault79
Copy link
Contributor

Hooray!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants