rules_appengine
rules_appengine copied to clipboard
Address post-merge comments on PR #71
- Quote bash variable when possible.
- Fix an undefined $root_dir
- Remove a redundant rm and let the defined trap handler do it.
- echo -e is needed when trying to display escape codes.
- Remove incorrect {{foo}} double curly braces.
@rogerhub Take a look please. I resisted the urge to change all of the $foo to ${foo} because I'm not sure if there is an official style. I stuck with the majority existing style.
Hi, is anyone still looking at this? The python .deploy commands are currently broken.