common-workflow-language / common-workflow-language/cwltool
double-adding requirements/hints on workflow steps
Open
- Dominant language
- Python
- Stars
- 376
- Forks
- 255
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 12
Description
`WorkflowStep.__init__()` adds the requirements to `loadingContext`, so that they will be applied to the embedded tool. However, when it calls the superclass `Process.__init__()` it assigns `self.requirements` from `loadingContext + tool["requirements"]` and as a result they are listed twice in the workflow step's `self.requirements`. This is harmless (all the requirements are still there and have the same relative priority) but it is confusing.
Contributor guide
Assessment
This issue has not been assessed yet.