Code review comment for lp://staging/~thumper/juju-core/get-environment

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

After speaking with fwereade, I drop my comment about the name.

On 2013/04/10 09:56:11, dimitern wrote:
> Looks good, but I have some questions.

> https://codereview.appspot.com/8597045/diff/1/cmd/juju/environment.go
> File cmd/juju/environment.go (right):

https://codereview.appspot.com/8597045/diff/1/cmd/juju/environment.go#newcode13
> cmd/juju/environment.go:13: type GetEnvironmentCommand struct {
> Why not just juju get instead?

https://codereview.appspot.com/8597045/

« Back to merge proposal