Skip to content

Commit a429479

Browse files
committed
Merge branch 'enh/assume_ready' of github.com:bpinsard/datalad-container into enh/assume_ready
2 parents cb661d6 + 901b604 commit a429479

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

datalad_container/containers_run.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,12 +176,14 @@ def __call__(cmd, container_name=None, dataset=None,
176176

177177
lgr.debug("extra_inputs = %r", extra_inputs)
178178

179-
if len(assume_ready) == 0:
179+
if not assume_ready:
180180
assume_ready = None
181181
elif len(assume_ready) == 1:
182182
assume_ready = assume_ready[0]
183183
elif "inputs" in assume_ready and "outputs" in assume_ready:
184184
assume_ready = "both"
185+
else:
186+
raise ValueError(f"Ended up with assume_ready={assume_ready!r} which must have not happened")
185187

186188
with patch.dict('os.environ',
187189
{CONTAINER_NAME_ENVVAR: container['name']}):

0 commit comments

Comments
 (0)