Fix dedup bam by apeltzer · Pull Request #190 · nf-core/eager

apeltzer · 2019-04-09T12:49:04Z

PR checklist

This comment contains a description of changes (with reason)
If you've fixed a bug or added code that should be tested, add tests!
If necessary, also make a PR on the nf-core/eager branch on the nf-core/test-datasets repo
Ensure the test suite passes (nextflow run . -profile test,docker).
Make sure your code lints (nf-core lint .).
Documentation in docs is updated
CHANGELOG.md is updated
README.md is updated

jfy133

Tested and you're good to go!

apeltzer added 2 commits April 9, 2019 14:44

changelog

d18d65a

Adding in dedup bam handling into a single place :)

6618632

apeltzer changed the base branch from master to dev April 9, 2019 12:49

apeltzer added 2 commits April 9, 2019 14:49

Bump for travisCI

0f4bfe4

Update changelog [skip ci]

d275fe9

apeltzer requested a review from maxibor April 9, 2019 13:12

jfy133 self-requested a review April 9, 2019 18:48

jfy133 approved these changes Apr 9, 2019

View reviewed changes

apeltzer merged commit 7769799 into nf-core:dev Apr 10, 2019

apeltzer deleted the fix-dedup-bam branch December 19, 2019 11:35