LGTM with one comment.
https://codereview.appspot.com/58970043/diff/1/provider/local/environprovider_test.go File provider/local/environprovider_test.go (right):
https://codereview.appspot.com/58970043/diff/1/provider/local/environprovider_test.go#newcode262 provider/local/environprovider_test.go:262: userOSErr: errors.New("oh noes"), You could just define userOSErr as string and construct an error from it in PatchValue below.
https://codereview.appspot.com/58970043/
« Back to merge proposal
LGTM with one comment.
https:/ /codereview. appspot. com/58970043/ diff/1/ provider/ local/environpr ovider_ test.go local/environpr ovider_ test.go (right):
File provider/
https:/ /codereview. appspot. com/58970043/ diff/1/ provider/ local/environpr ovider_ test.go# newcode262 local/environpr ovider_ test.go: 262: userOSErr: errors.New("oh
provider/
noes"),
You could just define userOSErr as string and construct an error from it
in PatchValue below.
https:/ /codereview. appspot. com/58970043/