Code review comment for lp:~themue/juju-core/go-environs-mode-errors

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

https://codereview.appspot.com/6715048/diff/1/environs/dummy/environs.go
File environs/dummy/environs.go (right):

https://codereview.appspot.com/6715048/diff/1/environs/dummy/environs.go#newcode494
environs/dummy/environs.go:494: env: e,
Doesn't look right. One environment state may be linked to from many
environment instances, and one instance is associated to one state.

Please have a look at the code you reverted to see how to do that
properly.

https://codereview.appspot.com/6715048/

« Back to merge proposal