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

Error message for numbers that should be strings in arguments could be clearer

Open
#1,009 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
376
Forks
255
Avg merge
2d 7h
Merged PRs (30d)
12

Description

```
baseCommand: tail
arguments:
- -n
- +2
- $(input.assembly_identifiers.path)
```
fails with an error because +2 is interpreted as a number instead of a string.
This is really confusing for new users.
```
Object
`....cwl` is
not valid because
tried `CommandLineTool` but
...:14:1: the `arguments` field is not valid because
tried array of but
...:16:5: item is invalid because
- tried string but
the value is not string
- tried Expression but
value is a int but expected a string
```

Could maybe be something.
```
Object
`....cwl` is
not valid because
tried `CommandLineTool` but
...:14:1: the `arguments` field is not valid because
tried array of but
...:16:5: item is invalid because
- tried string but
the value is not string
- tried Expression but
value is a int but expected a string
- '+2' looks like a number, try quoting
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.