LGTM modulo the comment below
https://codereview.appspot.com/8086045/diff/1/state/api/params/params_test.go File state/api/params/params_test.go (right):
https://codereview.appspot.com/8086045/diff/1/state/api/params/params_test.go#newcode55 state/api/params/params_test.go:55: Series: "series", // E.g. precise. Can we change the test so instead of using comments for "precise" and "wordpress" we instead have the test output show this?
https://codereview.appspot.com/8086045/
« Back to merge proposal
LGTM modulo the comment below
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go params/ params_ test.go (right):
File state/api/
https:/ /codereview. appspot. com/8086045/ diff/1/ state/api/ params/ params_ test.go# newcode55 params/ params_ test.go: 55: Series: "series", // E.g. precise.
state/api/
Can we change the test so instead of using comments for "precise" and
"wordpress" we instead have the test output show this?
https:/ /codereview. appspot. com/8086045/