Code review comment for lp:~canonical-platform-qa/snappy-ecosystem-tests/snapd-staging-build-script

Revision history for this message
Omer Akram (om26er) wrote :

I updated the approach based on some of the suggestions in your paste. I still kept a class that inherits from pylxd.models.Container as we want to extend a bit of functionality of that class.

I have also addressed some of your concerns as well.

Note: there is an outstanding issue with inheritance of Container class, I have sent a query to pylxd upstream developers for that.

« Back to merge proposal