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/ diff/1/ state/api/ uniter/ unit.go uniter/ unit.go (right):
File state/api/
https:/ /codereview. appspot. com/85670046/ diff/1/ state/api/ uniter/ unit.go# newcode445 uniter/ unit.go: 445: return nil, fmt.Errorf( "expected one Results) )
state/api/
result, got %d", len(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/