Code review comment for lp:~nskaggs/juju-ci-tools/add-sane-args-defaults

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

> However, in the case of
> someone wanting to run the tests locally, they might not particularly care
> beyond "use the juju I have".

Who is that person? Are they a juju developer?

If so, I think they are probably
1. writing a test for a feature they have written, or
2. running a test to make sure their changes don't break it

In both cases, they most likely have a custom binary that they want to test against.

So I still don't see who you're thinking would want to omit the path to the juju binary.

« Back to merge proposal