Code review comment for ~mertkirpici/juju-lint:lp/1989575

Revision history for this message
Martin Kalcok (martin-kalcok) wrote :

overall LGTM. just a small note. You can make multiple tox environments share the same "virtual environment folder" to cut down on venv duplicity. For example how "format-code" shares "lint" venv in juju-verify [0]. Same could be applied here.

[0] https://github.com/canonical/juju-verify/blob/237fea58fd579279121f476d244bacbd1992355c/tox.ini#L43

« Back to merge proposal