Code review comment for lp:~themue/pyjuju/go-state-remove-relation

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

LGTM, thanks. Just a trivial before submitting:

https://codereview.appspot.com/6250076/diff/17003/state/state.go
File state/state.go (right):

https://codereview.appspot.com/6250076/diff/17003/state/state.go#newcode377
state/state.go:377: // TODO: Improve high-level errors, no low-level
passing.
Please move this to within the function. Doesn't make sense to have it
exposed in the API documentation.

https://codereview.appspot.com/6250076/

« Back to merge proposal