Closed atrigila closed 2 weeks ago
nf-core pipelines lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit ce524ce
+| ✅ 300 tests passed |+
#| ❔ 6 tests were ignored |#
!| ❗ 4 tests had warnings |!
Looks great, love the new nf-test implementation, so much simple. Regarding @pinin4fjords comment about sharding, I also think it is easier to understand the tests, but I'm curious to know if we use sharding to make tests run faster or to save costs. If it is the latter, isn't it better to have it, even if reading the tests is a little bit harder?
As I understand it the sharding was introduced to bypass GitHub's limits on the numbers of independent tests, not out of any consideration to costs. In any case, my time is more valuable than computer time, so I'd like to save myself squinting at inscrutable test names.
@pinin4fjords done :) thank you!
Closes #61
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).