Closed nh13 closed 3 months ago
master
branch :x:base
to dev
Hi @nh13,
It looks like this pull-request is has been made against the nh13/fastquorum master
branch.
The master
branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to master
are only allowed if they come from the nh13/fastquorum dev
branch.
You do not need to close this PR, you can change the target branch to dev
by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.
Thanks again for your contribution!
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit ad28178
+| ✅ 169 tests passed |+
#| ❔ 8 tests were ignored |#
!| ❗ 1 tests had warnings |!
There's a bug in `samtools merge --template-coordinate' that causes the output not to be correctly sorted. Here, we re-sort via fgbio SortBam to ensure the output is properly sorted.
fgbio SortBam can be removed when the fix to the samtools bug is released.
See: https://github.com/samtools/samtools/pull/2062
PR checklist
nf-core lint
).nf-test test main.nf.test -profile test,docker
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).