Closed danilodileo closed 6 months ago
nf-core lint
overall result: Failed :x:Posted for pipeline commit daa9dce
+| ✅ 180 tests passed |+
#| ❔ 6 tests were ignored |#
!| ❗ 2 tests had warnings |!
-| ❌ 10 tests failed |-
Hi, there is a recent problem with nf-core template & most recent nextflow version. I'll resolve that after https://github.com/nf-core/ampliseq/pull/702 is done. So failing tests are expected!
Also remove the entry from nextflow_schema.json please.
Hi, there is a recent problem with nf-core template & most recent nextflow version. I'll resolve that after #702 is done. So failing tests are expected!
Also remove the entry from nextflow_schema.json please.
Hi Daniel, so can we just merge even if the errors are there?
so can we just merge even if the errors are there?
Yes, I can merge that. Just let me know and I'll do it.
so can we just merge even if the errors are there?
Yes, I can merge that. Just let me know and I'll do it.
It's ready to be merged!
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).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).Phytoref is included in PR2 5.0.0 so I removed it from DADA references option.