Commit 179742d8 authored by Aidan Richard Wiederhold's avatar Aidan Richard Wiederhold
Browse files

Merge branch 'aiwieder/local_filetype' into 'master'

Check if filetype is not None before checking if it matches an expected output.

See merge request !52
parents 7e658328 4a59afeb
Pipeline #3880243 passed with stage
in 4 minutes and 56 seconds
......@@ -162,10 +162,11 @@ def recursively_create_input(production_name, job_name, filetype, prod_data):
)
output_filetypes = [x.lower() for x in prod_data[job_name]["output"]]
if filetype.lower() not in output_filetypes:
raise NotImplementedError(
f"Looking for {filetype} but {job_name} only produces {output_filetypes}"
)
if filetype:
if filetype.lower() not in output_filetypes:
raise NotImplementedError(
f"Looking for {filetype} but {job_name} only produces {output_filetypes}"
)
# Run the job the desired job depends on
result = logging_subprocess_run(
......@@ -222,7 +223,7 @@ def resolve_input_overrides(prod_data, job_name):
)
else:
raise NotImplementedError(
"Input requires either a bookkeeping location or a previous job name"
"Input requires either a bookkeeping location, transform ids or a previous job name"
)
return override_input, output_filetype
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment