-
Notifications
You must be signed in to change notification settings - Fork 421
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 2.7.1 #374
Conversation
Bump sarek to 3.0dev
Merge from base
Retry tasks with exit code 247 for added memory
add option for controlfreec
Tumor-only mode for Mutect2 and MSIsensor
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.
Looks pretty good. I think some things ar emissing fom the Changelog. With regards to #378 Do you still want more tests before releasing?
<3
I did not pay attention to CHANGELOG lately, I'll make a PR for that.
Yeah, but not specifically for #378 only I think @ggabernet was having issues (#380) which I think are now fixed in this version. |
Since this release is still DSL1, I wanted to put |
Doesn't seem like you have much of a choice, do you? |
Yeah, it seems like so |
@nf-core/sarek Can I get approvals so that I can release (I can do it today, tomorrow or Thursday, but not Friday ;-)) |
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.
Looks good to me 🎉 , pending the CI tests passing
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.
They look good to me!
PR checklist
scrape_software_versions.py
nf-core lint .
).nextflow run . -profile test,docker
).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).