The safedata_zenodo generate_xml subcommand is broken - the function returns a string and error handling is not implemented correctly, so it doesn't follow the error, response pattern of the other subcommands. That's fine since it the function doesn't require a remote response, but it does need handling.
This could potentially be a separate entry point - safedata_describe? It feels rather out of place in the zenodo tools.
The
safedata_zenodo generate_xml
subcommand is broken - the function returns a string and error handling is not implemented correctly, so it doesn't follow theerror, response
pattern of the other subcommands. That's fine since it the function doesn't require a remote response, but it does need handling.This could potentially be a separate entry point -
safedata_describe
? It feels rather out of place in thezenodo
tools.