-
Notifications
You must be signed in to change notification settings - Fork 36
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
Prepare release 1.0.0 #53
Conversation
Dev merge template tools 2.0.1
Important! Template update for nf-core/tools v2.1
|
@maxulysse - any idea why some of the tests (push) are failing? |
No idea, but I noticed this in other pipelines as well, I'm asking on Slack |
@maxulysse Now rest all are completed, except one. It remains "waiting.." for a long time, anything wrong! |
Oh yes, we need to update the required tests in the settings, let me do that... |
Done. So now, let's ask for the final reviews |
Thanks! Who will do the final reviews? I think we need two approving reviews.. |
I have had a quick go-through as I am away this weekend, I will continue next week but it looks great in general! All my comments are relatively small. As a general comment on testing, its great to see the major pipeline modes tested as part of the CI, but if you have time I would go further. There seems to be some execution pathways that aren't covered with tests. |
Addressed the review comments
A step towards the first release of the pipeline.
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).