LGTM. Should log a couple of bugs: one to investigate why the quotes are lost, and one one to fix the testing situation.
https://codereview.appspot.com/14494054/diff/4001/environs/cloudinit/cloudinit_test.go File environs/cloudinit/cloudinit_test.go (right):
https://codereview.appspot.com/14494054/diff/4001/environs/cloudinit/cloudinit_test.go#newcode123 environs/cloudinit/cloudinit_test.go:123: // NOTE: this is terrible, only want to test part of the results... Indeed :(
https://codereview.appspot.com/14494054/
« Back to merge proposal
LGTM. Should log a couple of bugs: one to investigate why the quotes are
lost, and one one to fix the testing situation.
https:/ /codereview. appspot. com/14494054/ diff/4001/ environs/ cloudinit/ cloudinit_ test.go cloudinit/ cloudinit_ test.go (right):
File environs/
https:/ /codereview. appspot. com/14494054/ diff/4001/ environs/ cloudinit/ cloudinit_ test.go# newcode123 cloudinit/ cloudinit_ test.go: 123: // NOTE: this is terrible,
environs/
only want to test part of the results...
Indeed :(
https:/ /codereview. appspot. com/14494054/