-
Notifications
You must be signed in to change notification settings - Fork 1
First Release #33
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?
First Release #33
Conversation
integrate divergent branches
Dev preprocessing
Updating Container Images + Improved File staging + MultiQC
bump version + cleanup + multiqc update
|
Fix for failing github actions workflow
mapo9
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.
great work! 🚀
I was focusing on typos and code alignment a little and found a few things
|
Could this error in the CI test be related to the CI Nextflow version? |
|
I think in the past, this was more often a problem of disk space in the runner. can you try to add a clean up step before you run the pipeline? |
fix incorrect file permissions
Fixes for the release
|
great work! |
famosab
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!!!
A few more things and then you got the first release 🥳
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.
This does not exist yet? :) But I think for a first release having a full_size test profile would be nice. The full_size test does not need to run in the CI just locally and then on AWS and is what is being displayed then on the website.
| // stage input files into the working directory | ||
| // if test profile then first data needs to be unzipped |
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.
You could also do that by checking the file extension. See https://github.com/nf-core/variantprioritization/blob/dev/subworkflows/local/reference_data/main.nf how I did it. But thats just a suggestion not something you need to do :)
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.
oh but that's a very nice idea :)!
| and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html). | ||
|
|
||
| ## v1.0dev - [date] | ||
| ## v1.0.0 - [date] |
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.
Date and a release name are missing :)
|
|
||
|
|
||
| // Load nf-core/lsmquant custom profiles from different institutions. | ||
| // TODO nf-core: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs |
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.
?
Co-authored-by: Famke Bäuerle <45968370+famosab@users.noreply.github.com>
|
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 3.3.2. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
Co-authored-by: Matthias Hörtenhuber <mashehu@users.noreply.github.com>
PR 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).