Closed tillenglert closed 3 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit fb3c4c2
+| ✅ 196 tests passed |+
#| ❔ 2 tests were ignored |#
!| ❗ 2 tests had warnings |!
@nf-core-bot fix linting
LGTM! Just a minor comment regarding the description :)
Thank you @skrakau!
Documentation and secrets implementation
This PR implements the ncbi params:
NCBI_KEY
andNCBI_EMAIL
as nextflow secrets, to obscure them in the pipeline outputs likeparams.yml
. The Documentation how to define the secrets on respective user devices is also appended.PR https://github.com/nf-core/metapep/pull/119 contains the tests needing entrez downloads for reference.
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).