Code review comment for lp:~fwereade/juju-core/uniter-relation-states

Revision history for this message
William Reade (fwereade) wrote :

https://codereview.appspot.com/85670046/diff/1/state/api/uniter/unit.go
File state/api/uniter/unit.go (right):

https://codereview.appspot.com/85670046/diff/1/state/api/uniter/unit.go#newcode445
state/api/uniter/unit.go:445: return nil, fmt.Errorf("expected one
result, got %d", len(results.Results))
On 2014/04/09 17:47:11, hduran wrote:
> This is purely aesthetic and not sure if valid for English but if got
%d prints
> a number, "one" should also be a number.

FWIW, style guides I've seen have suggested that you should always spell
out numbers below N, for varying N. Being consistent works for me
though; done.

https://codereview.appspot.com/85670046/

« Back to merge proposal