Code review comment for lp:~mariosplivalo/juju-deployer/juju-deployer

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

The original option is semantically confusing, if you pass --local-mods it should means: enable local modifications and it should be True by default, also the destination variable should be called dest="local_mods" for reflecting this.

Mario, Does that makes sense for you? If does, please modify the code accordingly.

review: Needs Fixing

« Back to merge proposal