Code review comment for lp:~adam-collard/charms/precise/landscape-client/upgrade-charm-hook

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

[3]

Yeah I'd do that too then, but you're call. It can be done separately.

[4]

Okay, I see the point. We're already depending on mocker on the server charm tho, and in general I think we shouldn't be wary of adding test dependencies when they make sense. Your call as well.

« Back to merge proposal