common-workflow-language / common-workflow-language/cwl-intro-gui-workshop

Hands on #1 uses an invalid File object.

Open
#2 2 comments 1 reaction 1 assignee Claimed by @manabuishii View on GitHub
Dominant language
Common Workflow Language
Stars
8
Forks
4
PR merge metrics
No merged PRs in 30d

Description

A hands on in p.19 uses "Intro - Create a CWL Workflow.pdf" as an input parameter.

However, this `File` object is not good example because the [spec](https://www.commonwl.org/v1.0/CommandLineTool.html#File) says:
> If the `path` contains POSIX shell metacharacters (`|`,`&`, `;`, `<`, `>`, `(`,`)`, `$`, , `\`, `"`, `'`, ``, ``, and ``) or characters not allowed for Internationalized Domain Names for Applications then implementations may terminate the process with a `permanentFailure`.

That is, this example may not be applicable to other platforms such as `cwltool`.
It would be nice if the example uses another file that does not contain any metacharacters.

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.