Code review comment for lp:~jimbaker/pyjuju/expose-provider-ec2

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

[1]

On an overlook, this feels nice. We just debated online about one detail that
would be good to get handled before this is merged: there's no reason to have
that logic inside classes. Let's talk further online if you want to debate
more on this.

review: Needs Fixing

« Back to merge proposal