common-workflow-language / common-workflow-language/cwltool

Pin nodejs version for InlineJavascriptRequirement

Open
#1,207 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Python
Stars
376
Forks
255
Avg merge
2d 7h
Merged PRs (30d)
12

Description

Hello!
I suggest to pin the version of the nodejs docker image (instead of `:latest`).
Now if `InlineJavascriptRequirement` is mentioned in the schema and `nodejs` is not installed locally, then `cwltool` downloads a docker image from the dockerhub. Let's pull a specific version, not the `:latest`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.