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
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/