-
Notifications
You must be signed in to change notification settings - Fork 6
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
Patch 1.2.1: Fix issue where pipeline fails on missing fields from NCBI datasets JSON #127
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.
Good effort with the linting, @tkchafin, especially adding documentation for all the functions. But unfortunately, black
disagrees and as you know, it's "opinionated" and "uncompromising" 😬 . We've got black
installed on the farm in the nf-core Conda environments. That'll make the script comply with the CI.
Then, since we want to make a 1.2.1 release directly with this bugfix, please bump the version and update these files:
- CHANGELOG.md
- nextflow.config
- CITATION.cff
and change the target branch tomain
(the "Run pipeline with test data" CIs now pass. That failure was transient) |
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.
Almost there !
Co-authored-by: Matthieu Muffato <[email protected]>
Co-authored-by: Matthieu Muffato <[email protected]>
Also note that currently we are still using |
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.
Changes look good for me, but can you wait a tiny bit before merging ? We have multiple Zenodo DOIs for our pipelines, and I'm in conversation with them about unifying the DOIs first
Merging this as the DOI is actually correct |
Closes #126
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).