Code review comment for lp:~nuclearbob/utah/isoclass

Revision history for this message
Javier Collado (javier.collado) wrote :

Looked at the changes and they look nice. Also provisioned a VM from scratch using this branch and worked fine. Thanks.

My only comment is that please try to avoid unrelated changes in merge requests that fix a bug. I small refactoring is nice, but I think that removing a whole file and adding a lot of documentation strings should be part of a separate merge request.

review: Approve

« Back to merge proposal