-
Notifications
You must be signed in to change notification settings - Fork 12
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
Template update #165
Template update #165
Conversation
…lign into template-update
|
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.
I made some style comments.
But mostly, I am not sure if the template update removed all your patch files from modules, so please make sure that everything is as it should be 🙂
After this is addressed, LGTM 🚀
subworkflows/local/utils_nfcore_multiplesequencealign_pipeline/main.nf
Outdated
Show resolved
Hide resolved
Thanks a lot for the review!! |
Co-authored-by: Júlia Mir Pedrol <[email protected]>
Co-authored-by: Júlia Mir Pedrol <[email protected]>
Co-authored-by: Júlia Mir Pedrol <[email protected]>
…/main.nf Co-authored-by: Júlia Mir Pedrol <[email protected]>
…equencealign into template-update
PR checklist
Template update PR