juju-core

Code review comment for lp:~menno.smits/juju-core/1302005-bootstrap_series

  1. 1302005-bootstrap_series
  2. Merge into trunk
Revision history for this message
William Reade (fwereade) wrote :

LGTM

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

https://codereview.appspot.com/94060044/diff/1/cmd/juju/bootstrap.go#newcode128
cmd/juju/bootstrap.go:128: }
Quibble quibble whine: I'd rather validate before setting on the
StringsValue.

https://codereview.appspot.com/94060044/

« Back to merge proposal