-
Notifications
You must be signed in to change notification settings - Fork 30
Release 3.1.0 #401
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
base: master
Are you sure you want to change the base?
Release 3.1.0 #401
Conversation
Co-authored-by: Maxime U Garcia <[email protected]>
…hcquant into template-merge-331
Add box plots to the MulitQC report
Aratz
left a comment
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.
Nice work!
I left a few comments but I think for this release the most important ones are to fix the citations.
The other comments concern mostly tests and readability, and could also be addressed in a later release, I leave it up to you how you want to keep track of and address them :)
rolivella
left a comment
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.
@jonasscheid very good job! Just only a few concerns and comments. :)
Co-authored-by: Adrien Coulier <[email protected]>
Co-authored-by: Adrien Coulier <[email protected]>
Apply reviewer comments
… to wait for conda lock file support
Temporarily drop conda and epicore testing due to unstable CI tests.
Final adjustments to 3.1.0 Release
AddedFastacolumn to the input samplesheet to enable sample-specific FASTA files #391FixedDependenciesMultiQCNf-coreEasyPQPPR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).