Clarify parameter reference: null and array length #64
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
Author
|
This PR updates the text of the specification and also adds tests for some incorrect usages of parameter references. |
mr-c
reviewed
Nov 24, 2020
tom-tan
reviewed
Nov 25, 2020
Contributor
Author
Member
|
@pvanheus not written down. There should be one of If no optional parts like The above needs to be added to the conformance doc 😂 |
70 tasks
mr-c
force-pushed
the
null_and_somearray_length
branch
from
32ac3b2 to
871afac
Compare
4 tasks
mr-c
approved these changes
Nov 26, 2021
kinow
pushed a commit
to kinow/cwl-v1.2
that referenced
this pull request
Oct 20, 2022
…anguage#64) * Address ambiguity on null and inputs.somearray.length in parameter reference * Clarify that the length field can sometimes apply to a non-array input
GlassOfWhiskey
pushed a commit
that referenced
this pull request
Nov 15, 2023
* Address ambiguity on null and inputs.somearray.length in parameter reference * Clarify that the length field can sometimes apply to a non-array input
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This aims to address common-workflow-language/common-workflow-language#761 at least on the documentation level. The associated test should be expanded to ensure that incorrect usages are not allowed (e.g.
$(null.something)).