Thank you both for the reviews of this long diff!
https://codereview.appspot.com/174790043/diff/1/quickstart/app.py File quickstart/app.py (left):
https://codereview.appspot.com/174790043/diff/1/quickstart/app.py#oldcode188 quickstart/app.py:188: if requires_sudo: On 2014/11/10 15:25:20, rharding wrote: > is this because of the new juju requirement?
Yes! We got rid of very old versions of juju requiring an explicit sudo on the command line.
https://codereview.appspot.com/174790043/
« Back to merge proposal
Thank you both for the reviews of this long diff!
https:/ /codereview. appspot. com/174790043/ diff/1/ quickstart/ app.py
File quickstart/app.py (left):
https:/ /codereview. appspot. com/174790043/ diff/1/ quickstart/ app.py# oldcode188 app.py: 188: if requires_sudo:
quickstart/
On 2014/11/10 15:25:20, rharding wrote:
> is this because of the new juju requirement?
Yes! We got rid of very old versions of juju requiring an explicit sudo
on the command line.
https:/ /codereview. appspot. com/174790043/