Closed atrigila closed 2 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit ff5ace9
+| ✅ 214 tests passed |+
#| ❔ 1 tests were ignored |#
!| ❗ 3 tests had warnings |!
That doesnt have any unforeseen side effects? In the past I sometimes had other tools using samtools indices that went awkward when being supplied with a CSI index. But ok, the tests run fine 👍🏻
I took a closer and it makes sense that the changes have no other impact. Looking at the subworkflow "BAM_SORT_STATS_SAMTOOLS" script: The both the .csi or .bai files that are outputed are then combined with the .bam files channel and then used as input for "BAM_STATS_SAMTOOLS" which produces 3 stat files as output which we are validating with md5sums in the tests. But besides this, the .csi or .bai files emitted by "SAMTOOLS_SORT" are not used for anything else afterwards.
Exactly :) thanks for having a look
Cool then :)
Fixes #132
PR checklist
nf-core lint
).nf-test test main.nf.test -profile test,docker
).CHANGELOG.md
is updated.