Code review comment for lp:~evarlast/juju-quickstart/which-juju

Revision history for this message
Francesco Banconi (frankban) wrote :

9 + if os.environ.get('JUJU', False):

This can be written as:

juju_command = os.getenv('JUJU', '').strip()
if not juju_command:
    juju_command = platform_support.get_juju_command(options.platform)

However, I'd like this logic to be placed in get_juju_command(), so that the function can still be potentially re-used in other parts of the code.

« Back to merge proposal