Code review comment for lp:~soren/nova/derootification

Revision history for this message
termie (termie) wrote :

i'd say skip making a local copy of ArgumentSerializer if we are only using it once, otherwise it either needs an additional newline before it or no newline before it.

the flag parser class needs additional newlines on either side of it.

please make the long line fit within 80 chars

other than those style nits, looks great :) Will approve once those are in.

review: Needs Fixing

« Back to merge proposal