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