Code review comment for lp:~dave-cheney/juju-core/102-cmd-init-context

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

If I were implementing it I think I would have gone for something more
like c.Config.Open(ctx) inside Run, but I can't come up with a really
convincing case for that being *better*. (OK, it'd be less work, but the
work's already done.) So: LGTM.

https://codereview.appspot.com/6532045/

« Back to merge proposal