Code review comment for lp://staging/~gary/juju-core/more-info

Revision history for this message
Tim Penhey (thumper) wrote :

LGTM modulo the comment below

https://codereview.appspot.com/8086045/diff/1/state/api/params/params_test.go
File state/api/params/params_test.go (right):

https://codereview.appspot.com/8086045/diff/1/state/api/params/params_test.go#newcode55
state/api/params/params_test.go:55: Series: "series", // E.g. precise.
Can we change the test so instead of using comments for "precise" and
"wordpress" we instead have the test output show this?

https://codereview.appspot.com/8086045/

« Back to merge proposal