Code review comment for lp://staging/~fwereade/juju-core/environs-tools-provisioning

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

LGTM

https://codereview.appspot.com/8727044/diff/1/environs/tools_test.go
File environs/tools_test.go (right):

https://codereview.appspot.com/8727044/diff/1/environs/tools_test.go#newcode585
environs/tools_test.go:585: var findBootstrapToolsTests = []struct {
nice, exhaustive tests!

https://codereview.appspot.com/8727044/diff/1/environs/tools_test.go#newcode607
environs/tools_test.go:607: info: "released cli: cli arch
ignored",
why is it ignored (both cases below) ?

https://codereview.appspot.com/8727044/

« Back to merge proposal