common-workflow-language / cwltool

Common Workflow Language reference implementation
https://cwltool.readthedocs.io/
Apache License 2.0
332 stars 230 forks source link

cwltool --validate doesn't check for InlineJavascriptRequirement when it should #1873

Open sjaenick opened 1 year ago

sjaenick commented 1 year ago

Hi,

cwltool 3.1.20230601100705

For a workflow with cwlVersion: 1.2 and some steps only executed conditionally via when, cwltool successfully validates the workflow even if InlineJavascriptRequirement is not specified.

sjaenick commented 1 year ago

Sample workflow e.g. at https://github.com/MGX-metagenomics/cwl/blob/master/metagenome-assembly-pipeline.cwl - cwltool --validate still reports success even if the InlineJavascriptRequirement is removed.