ReproNim / reproman

ReproMan (AKA NICEMAN, AKA ReproNim TRD3)
https://reproman.readthedocs.io
Other
24 stars 14 forks source link

BF: orchestrators: Don't try to get outputs if ref merge fails #529

Closed kyleam closed 4 years ago

kyleam commented 4 years ago

As of cdce12249 (ENH: orchestrators(datalad-pair): Do a more targeted update, 2020-04-09), we first try to merge in the job ref, and if that fails, update() isn't called. Set failure=True so that we do not try to get output files with a working tree that has not been updated.

Reported-by: @yarikoptic https://github.com/ReproNim/reproman/pull/527#discussion_r435427464

yarikoptic commented 4 years ago

Travis is really green not yellow. Thank you!