-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Linting for release #29
Conversation
|
Hi @muffato Thanks for writing this up.
Can't be fixed
|
Can you try putting the
Here are all the places where we have the version number: https://github.com/sanger-tol/genomenote/pull/127/files (exclude the Python script). Make sure all those files are updated in this PR here. |
Super ! I fixed the CI error, the last linting error and a few warnings. All good now |
Up to you @DLBPointon to Approve and merge |
Thanks for all the help with this @muffato ! |
This is a PR to solve the issues flagged by
nf-core lint
in #28 . I've started fixing a few, but @DLBPointon can you check the other ones ?I've added them back b9d2630
The parser was confused by the whitespace 68b369b
Only
modules/local/reformat_intersect.nf
needs fixingprocess_tiny
is not recognised. Switched toprocess_single
f71aeb2It looks like
nf-core lint
didn't like the double quotes 501572cThis is
get_largest_scaff.nf
You need to bump the version number in
nextflow_config
and other filesI think these ones don't need fixing, but please confirm:
nf-core lint
seems unable to parse / verifydocker.io/ubuntu:20.04
The versions being used have been validated in production. We don't want to change them now.
That's because it expects
""" ... """
but you use$/ ... /$
. I guess that's on purpose to simplify the handling of$
?I think it's complaining that the Singularity and Docker versions mismatch
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).