Code review comment for lp:~nskaggs/juju-ci-tools/add-snap-to-default-juju-args

Revision history for this message
Aaron Bentley (abentley) wrote :

You don't need to do more, you need to do less. This change produces only 5 test failures.
=== modified file 'utility.py'
--- utility.py 2017-01-19 04:47:42 +0000
+++ utility.py 2017-02-13 20:19:51 +0000
@@ -338,7 +338,7 @@
                         help='Full path to the Juju binary. By default, this'
                         ' will use $GOPATH/bin/juju or /usr/bin/juju in that'
                         ' order.',
- default=_generate_default_binary())
+ default=None)

 def add_basic_testing_arguments(parser, using_jes=False, deadline=True,

« Back to merge proposal