common-workflow-language / common-workflow-language/common-workflow-language

deprecate and then remove CWL Expressions in Process.{inputs,outputs}.{format,secondaryFiles}

Open
#865 4 comments 0 reactions 0 assignees View on GitHub
Dominant language
Common Workflow Language
Stars
1.5k
Forks
193
PR merge metrics
No merged PRs in 30d

Description

(inspired by working on https://github.com/common-workflow-language/cwl-utils/pull/5)

Allowing CWL expressions in `Process.{inputs,outputs}[].{format,secondaryFiles}` makes a highly dynamic signature for `Process`es.

Now that CWL v1.1 has optional `secondaryFiles` I suspect there are no more use cases for allowing CWL expressions in `secondaryFiles`

For `format` I don't see an argument for allowing CWL Expressions in the `inputs` and for `outputs` it would be better to list all possible `format`s (which is currently allowed) and pick/override the `format` inside `outputEval`.

- [ ] use the `view.commonwl.org` archive to examine how users use CWL expressions in these fields to confirm this idea.

If confirmed:
- [ ] deprecate in `cwlVersion: v1.2`
- [ ] remove from `cwlVersion: v2.0`

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.