Closed d4straub closed 1 year ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 472534a
+| ✅ 152 tests passed |+
#| ❔ 3 tests were ignored |#
!| ❗ 2 tests had warnings |!
Thanks for the approval!
If adding things to the MultiQC is relatively little work, maybe one can have things in both places? MultiQC html is mostly for human consumption, while report tables are more for programs.
I meant MultiQC or https://nf-co.re/ampliseq/dev/docs/output#pipeline-summary-report (which is also a html). In MultiQC the same/a subset of if statements as in https://github.com/nf-core/ampliseq/blob/6a1adc249dca6a696e4c87af9d4232bba9570230/workflows/ampliseq.nf#L745-L803 would be needed I think. Thats just a maintenance nightmare, so I would like to avoid multiple of that if statements.
This is just a minimal update of the MultiQC method section. It could be extended to a complete section with all tools and citations. However I am not sure whether I would like to have that rather in the pipeline summary report or in MultiQC. Anyway, taking the shortcut here and modify only whats absolutely required.
PR checklist
nf-core lint
).nextflow run . -profile 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).