Code review comment for lp:~jaypipes/glance/use-optparse

Revision history for this message
Rick Harris (rconradharris) wrote :

I'll throw in my 2c here.

I personally agree with Devin, argparse is a great library, does some things that optparse can't, and is easy enough to include as a dependency. So I'd be a +1 on moving to argparse.

That said, I'm totally cool with going with optparse for now and then moving to argparse when 2.7 becomes more standard.

No biggie.

review: Approve

« Back to merge proposal