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

Devin Carlen (devcamcar) wrote :

It doesn't, of course. :)

It's just one of those things - if it is easy to use argparse over optparse, then we get some future proofing which is always good.

It looks like it would be pretty trivial to make the change, but I'll leave it up to you at the end of the day.

Either way I'll approve this for now, if you decide to go with argparse it can be a separate merge.

review: Approve

« Back to merge proposal