Closed jfy133 closed 5 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 9f4aa1d
+| ✅ 210 tests passed |+
#| ❔ 3 tests were ignored |#
!| ❗ 5 tests had warnings |!
@nf-core-bot fix linting
Looks good, just one comment (according to https://nf-co.re/docs/contributing/pipeline_release_review_guidelines): some contributors are missing in the README (at least @CarsonJM as far as I can see)
The scripts should get licenses at some point
Yup, Maxime pointed this out too :grimacing: - need to find original authors or come up with best way to put e.g. nf-core communtiy or something, but will take a bit of time. Made an issue already #570
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).