LGTM with a couple of fixes
https://codereview.appspot.com/40370047/diff/1/worker/uniter/context.go File worker/uniter/context.go (right):
https://codereview.appspot.com/40370047/diff/1/worker/uniter/context.go#newcode214 worker/uniter/context.go:214: // into ctx to execute jujuc tools. Doc string is out of date
https://codereview.appspot.com/40370047/diff/1/worker/uniter/context_test.go File worker/uniter/context_test.go (right):
https://codereview.appspot.com/40370047/diff/1/worker/uniter/context_test.go#newcode733 worker/uniter/context_test.go:733: func (s *RunCommandSuite) GetHookContext(c *gc.C) *uniter.HookContext { this doesn't need to be exported
https://codereview.appspot.com/40370047/
« Back to merge proposal
LGTM with a couple of fixes
https:/ /codereview. appspot. com/40370047/ diff/1/ worker/ uniter/ context. go uniter/ context. go (right):
File worker/
https:/ /codereview. appspot. com/40370047/ diff/1/ worker/ uniter/ context. go#newcode214 uniter/ context. go:214: // into ctx to execute jujuc tools.
worker/
Doc string is out of date
https:/ /codereview. appspot. com/40370047/ diff/1/ worker/ uniter/ context_ test.go uniter/ context_ test.go (right):
File worker/
https:/ /codereview. appspot. com/40370047/ diff/1/ worker/ uniter/ context_ test.go# newcode733 uniter/ context_ test.go: 733: func (s *RunCommandSuite)
worker/
GetHookContext(c *gc.C) *uniter.HookContext {
this doesn't need to be exported
https:/ /codereview. appspot. com/40370047/