Code review comment for lp:~sebastiken/anybox.recipe.openerp/group_standalone_addons

Revision history for this message
Georges Racinet (gracinet) wrote :

Thanks for the contribution !

The idea sounds nice, and this may even become the successor of the existing magic for standalones
(which took us a long while to really work and is a source of problems in itself).
About this, see also https://bugs.launchpad.net/anybox.recipe.openerp/+bug/1214832 which I didn't have the courage to start yet.

But it'll take me a while to truly evaluate it. I wonder in particular if all the features that need to instanciate a VCS subclass will support it out of the box with so simple a diff.

Also, like most features, it'll need its own unit tests.

« Back to merge proposal