Code review comment for lp://staging/~rogpeppe/juju-core/293-status-life

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

LGTM, although please merge trunk and change Series in machineStatus as
agreed (not omitempty).

https://codereview.appspot.com/8852043/diff/1/cmd/juju/status.go
File cmd/juju/status.go (right):

https://codereview.appspot.com/8852043/diff/1/cmd/juju/status.go#newcode137
cmd/juju/status.go:137: status.Life,
did gofmt do that? it looks surprisingly ugly..

https://codereview.appspot.com/8852043/

« Back to merge proposal