-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add option to remove conda-env name #45
Comments
It looks like it's |
This is the naming convention we use to generate the tag for an image. |
Could we do something like: if( condaFile && (prefix=guessCondaRecipeName(condaFile)) ) {
+ tag = "${repo : normaliseTag(prefix) + "--" ? "" }${id}"
- tag = "${normaliseTag(prefix)}--${id}"
} |
What's the rationale of stripping the package name? it makes much more readble |
I agree it does! Just the optionality because it's redundant if there's a freeze repo. Probably not a priority though, just a few repeated words. |
This will be solved by seqeralabs/wave#395 |
Closing in favour of #69 |
Getting this:
nf-core/modules/bowtie/build:bowtie_build--7d5920d114506960
And I'd love to see
nf-core/modules/bowtie/build:7d5920d114506960
Not a top priority just a thought.
The text was updated successfully, but these errors were encountered: