Closed nh13 closed 3 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit f086100
+| ✅ 170 tests passed |+
#| ❔ 8 tests were ignored |#
!| ❗ 1 tests had warnings |!
@SPPearce since while we're waiting for https://github.com/samtools/samtools/issues/2090, lets merge this, then we can create an issue to track the samtools issue
I think we want this in the next release. I’ll look at this in the next few days
Are we ready for a release now then @nh13 ?
@SPPearce I think so, is that something you want to get started on, if not, I can take a look this weekend.
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).