-
Notifications
You must be signed in to change notification settings - Fork 20
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
Small updates to schema JSON to remove false and "None" defaults. #302
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In principle fine - @jasmezz can you double check that the prokka scientific notation to decimal is correct for prokka?
Yes, I tested that and works. Just Bakta DB download seems flaky recently, often "too many download attempts" error in CI tests. Let's hope it is willing to succeed today. |
The schema currently specifies a number of
String
parameters that have "None" as the default value. This default will be used verbatim in Nextflow pipelines and cause errors. They are removed in this PRPR 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).