Skip to content

Commit e911d23

Browse files
authored
Merge pull request #21 from common-workflow-language/inherited-requirements
Clarify inherited requirements
2 parents 3cf1664 + ce3c91a commit e911d23

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

Workflow.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ $graph:
4848
4949
## Changelog
5050
51+
* Clarify how Requirements are inherited by step processes
5152
* Document explicitly that CWL allows pass-through of workflow inputs to outputs
5253
* Default values for some fields are now expressed in the schema
5354
* When defining record types, fields of type `File` may now include

concepts.md

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -215,9 +215,14 @@ object document as an array of requirements under the field name
215215
should be combined with any requirements present in the corresponding Process
216216
as if they were specified there.
217217

218-
Requirements are inherited. A requirement specified in a Workflow applies
219-
to all workflow steps; a requirement specified on a workflow step will
220-
apply to the process implementation of that step and any of its substeps.
218+
Requirements specified in a parent Workflow are inherited by step processes
219+
if they are valid for that step. If the substep is a CommandLineTool
220+
only the `InlineJavascriptRequirement`, `SchemaDefRequirement`, `DockerRequirement`,
221+
`SoftwareRequirement`, `InitialWorkDirRequirement`, `EnvVarRequirement`,
222+
`ShellCommandRequirement`, `ResourceRequirement` are valid.
223+
224+
*As good practice, it is best to have process requirements be self-contained,
225+
such that each process can run successfully by itself.*
221226

222227
If the same process requirement appears at different levels of the
223228
workflow, the most specific instance of the requirement is used, that is,

0 commit comments

Comments
 (0)