[ARVADOS] updated: 9640930c3934344ab64c92a5c86c1f7488a4de42
Git user
git at public.curoverse.com
Fri Nov 11 16:23:35 EST 2016
Summary of changes:
sdk/cli/bin/arv-run-pipeline-instance | 2 ++
1 file changed, 2 insertions(+)
via 9640930c3934344ab64c92a5c86c1f7488a4de42 (commit)
from d0bc46ce842e7fdb71c25cc32caef0afd209c9ec (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit 9640930c3934344ab64c92a5c86c1f7488a4de42
Author: Tom Clegg <tom at curoverse.com>
Date: Fri Nov 11 16:19:58 2016 -0500
3486: Fix ignored "required: false" in pipeline template. refs #3486 refs #10470
diff --git a/sdk/cli/bin/arv-run-pipeline-instance b/sdk/cli/bin/arv-run-pipeline-instance
index bcb11d1..960d784 100755
--- a/sdk/cli/bin/arv-run-pipeline-instance
+++ b/sdk/cli/bin/arv-run-pipeline-instance
@@ -380,6 +380,8 @@ class WhRunPipelineInstance
value = params[parametername.to_s]
elsif parameter.has_key?(:default)
value = parameter[:default]
+ elsif [false, 'false', 0, '0'].index(parameter[:required])
+ value = nil
else
errors << [componentname, parametername, "required parameter is missing"]
next
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list