-
Notifications
You must be signed in to change notification settings - Fork 3
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
Release 1.0.0 #8
Conversation
Co-authored-by: Matthias Hörtenhuber <[email protected]>
As suggested in PR#9
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Improve the pipeline workflow graphic/map
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
I have a couple of documentation and credit updates that I would like to make, and potential improvements to the MultiQC output. Shall I wait first for the 1.0.0 release, or can I bring the changes in via PR without invalidating the first approval ? |
bring them in and ping me, so I can give you a quick second one 😉 |
Hi @mashehu I have pushed three commits related to MultiQC, acknowledgements and documentation. On our side we are now ready for 1.0.0 release! (And I will be absent the next two weeks by the way). |
Hello everybody, I will be back from holidays next week. Please let me know what I can do to complete the release of the pipeline. |
You can ask for a review on slack in the #request-review channel |
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.
Just one comment to check.
I think someone with admin rights would also need to remove the old required check |
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.
LGTM!
Thanks! Can I press the Merge button? |
Hello,
this is the first release of the pairgenomealign pipeline!
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).