-
Notifications
You must be signed in to change notification settings - Fork 5
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
Change repo to nf-core/actions #22
Comments
What is the rationale to reinstall e.g. Nextflow frequently (caching aside) over running those actions requiring it in a Docker container that already has the required dependencies installed? |
Okay, yes, that's reasonable - I wasn't aware that we are actually testing containerized tasks somewhere. Thanks for letting me know and the effort to provide me with precise links to docs and code! Fascinating to see how Nextflow handles the Docker execution in that case! |
I was debating setting this up for the new nf-test CI workflow that was being worked on. But I think these will sadly need to be separate actions. https://github.com/tj-actions Otherwise we'll have a new version every time any other actions have a release. I think we back off this and just keep things simple with separate repos. |
Originally posted by @emiller88 in #19 (comment)
The text was updated successfully, but these errors were encountered: