[x] All new jobs, commands, executors, parameters have descriptions (not needed)
[x] Examples have been added for any significant new features (not needed)
[x] README has been updated, if necessary (not needed)
Motivation, issues
Fixes #333
The change is needed to update the ${HOME}/.docker/config.json file to with multiple ECR entries. The jq command currently fails when attempting to add a second ECR entry.
Description
Removes double quotes around the $jq_flag so that it is properly handled when it is empty. With the double quotes, the jq command is interpreting the empty string as a parameter and then failing to find the config file because of how it is parsing the flags.
I also fix the indentation of the jq command so that the staring matches the level of the if statement above it and indent the additional parts of the jq command a couple extra spaces.
Checklist
Motivation, issues
Fixes #333
The change is needed to update the
${HOME}/.docker/config.jsonfile to with multiple ECR entries. Thejqcommand currently fails when attempting to add a second ECR entry.Description
Removes double quotes around the
$jq_flagso that it is properly handled when it is empty. With the double quotes, thejqcommand is interpreting the empty string as a parameter and then failing to find the config file because of how it is parsing the flags.I also fix the indentation of the
jqcommand so that the staring matches the level of theifstatement above it and indent the additional parts of thejqcommand a couple extra spaces.