clean up syntax #105
clean up syntax #105
Conversation
mr-c
requested review from
tobyhodges and
tom-tan
and removed request for
tom-tan
| requirements: | ||
| - class: InlineJavascriptRequirement | ||
| - class: InitialWorkDirRequirement | ||
| InitialWorkDirRequirement: |
There was a problem hiding this comment.
This syntax is not valid in v1.0.2 because the spec says that requirements should be an array.
There was a problem hiding this comment.
In the spec itself a mapping using the value of the class field as the key is allowed: https://github.com/common-workflow-language/common-workflow-language/blob/master/v1.0/Process.yml#L811 but I see that it isn't rendering in the HTML version: http://www.commonwl.org/v1.0/CommandLineTool.html#CommandLineTool I also reviewed all 10 captures of this page in archive.org across 2016 & 2017 and the same rendering bug is present.
@tetron Any ideas?
There was a problem hiding this comment.
I've opend an issue about this at common-workflow-language/schema_salad#167 and I think it is rather important to fix.
There was a problem hiding this comment.
@mr-c I got it. Thanks for the information!
3 participants
No description provided.