Code review comment for lp:~frankban/juju-quickstart/new-auth-api-endpoint

Revision history for this message
Martin Hilton (martin-hilton) wrote :

LGTM: No QA

https://codereview.appspot.com/199490043/diff/1/quickstart/app.py
File quickstart/app.py (right):

https://codereview.appspot.com/199490043/diff/1/quickstart/app.py#newcode283
quickstart/app.py:283: def get_env_uuid_or_none(env_name):
Is it really necessary to have or_none in the name of this function?

https://codereview.appspot.com/199490043/

« Back to merge proposal