Code review comment for lp:~thumper/juju-core/juju-errors

Revision history for this message
Andrew Wilkins (axwalk) wrote :

LGTM with a minor renaming

https://codereview.appspot.com/93410043/diff/1/provider/openstack/storage.go
File provider/openstack/storage.go (right):

https://codereview.appspot.com/93410043/diff/1/provider/openstack/storage.go#newcode12
provider/openstack/storage.go:12: coreerrors "github.com/juju/errors"
s/coreerrors/jujuerrors/

https://codereview.appspot.com/93410043/diff/1/state/apiserver/client/destroy_test.go
File state/apiserver/client/destroy_test.go (right):

https://codereview.appspot.com/93410043/diff/1/state/apiserver/client/destroy_test.go#newcode9
state/apiserver/client/destroy_test.go:9: coreerrors
"github.com/juju/errors"
ditto
(and anywhere else coreerrors exists)

https://codereview.appspot.com/93410043/

« Back to merge proposal