nextflow-io / nextflow

A DSL for data-driven computational pipelines
http://nextflow.io
Apache License 2.0
2.61k stars 606 forks source link

Charliecloud: registry, docs #5018

Closed nschan closed 1 month ago

nschan commented 1 month ago

Hi,

I noticed that my previous attempt (https://github.com/nextflow-io/nextflow/pull/4879) at making charliecloud handle the registry does not work. This PR fixes this by following the general approach of other container engines: letting normalizeDockerImageName handle this. I also realized that my previous PR added three new options for charliecloud, but these were not documented, so this also updates the docs accordingly. This is identical to #5017, except that here I signed off the commits.

netlify[bot] commented 1 month ago

Deploy Preview for nextflow-docs-staging canceled.

Name Link
Latest commit 8f02af8e1742731d0f638ba227763048914e4a72
Latest deploy log https://app.netlify.com/sites/nextflow-docs-staging/deploys/664e07f73950060008d949f1
pditommaso commented 1 month ago

Ok, thanks