Closed nschcolnicov closed 1 month ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 8e54dbc
+| ✅ 231 tests passed |+
#| ❔ 1 tests were ignored |#
!| ❗ 3 tests had warnings |!
Is this PR addressing #400 or a subset of #400?
Is this PR addressing #400 or a subset of #400?
I'm not sure I understand this question, can you elaborate?
Are we addressing the whole scope of #400 or some sub-tasks here (in this PR)?
Are we addressing the whole scope of #400 or some sub-tasks here (in this PR)?
- [x] Address the TODOs left in the pipeline.
- [x] Check variables, add "ch" to channel and "val" to value variables
- [x] Make sure that variable names remain consistent across workflows.
- [x] Assess which emitted files from subworkflows are actually being used.
- [x] Replace any dsl1 or groovy funcitons which can be replaced with nextflow DSL2 functions.
- [x] Reviewing parameters modules and subworkflows to check there are no unused items.
Ah, I see, yes we adressing all of the items in the issue, we could also add, I will also add, reviewing parameters modules and subworkflows to check there are no unused items.
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).