https://codereview.appspot.com/14279044/diff/1/environs/bootstrap/bootstrap.go File environs/bootstrap/bootstrap.go (left):
https://codereview.appspot.com/14279044/diff/1/environs/bootstrap/bootstrap.go#oldcode96 environs/bootstrap/bootstrap.go:96: // TODO(rog) this feels like a layering violation - providers On 2013/10/03 03:21:00, axw1 wrote: > Is this TODO no longer applicable? Seems to be...
I took it out because everything expects it to be there. It is how bootstrap checks to see if the environment is already bootstrapped. I think for consistency across providers this comment no longer makes sense.
https://codereview.appspot.com/14279044/
« Back to merge proposal
https:/ /codereview. appspot. com/14279044/ diff/1/ environs/ bootstrap/ bootstrap. go bootstrap/ bootstrap. go (left):
File environs/
https:/ /codereview. appspot. com/14279044/ diff/1/ environs/ bootstrap/ bootstrap. go#oldcode96 bootstrap/ bootstrap. go:96: // TODO(rog) this feels like a
environs/
layering violation - providers
On 2013/10/03 03:21:00, axw1 wrote:
> Is this TODO no longer applicable? Seems to be...
I took it out because everything expects it to be there.
It is how bootstrap checks to see if the environment
is already bootstrapped. I think for consistency across
providers this comment no longer makes sense.
https:/ /codereview. appspot. com/14279044/