Code review comment for lp:~trapnine/maas/fix-1564971

Revision history for this message
Mike Pontillo (mpontillo) wrote :

Looks good; I see some minor issues that I won't block you on, mostly regarding docstrings and comments. (see comments below.) Also, the major function in this branch is a little long and could stand to be refactored to reduce its complexity, IMO.

review: Approve

« Back to merge proposal