Code review comment for ~pfsmorigo/ubuntu-cve-tracker:pfsmorigo/add_endpont_argument

Revision history for this message
Steve Beattie (sbeattie) wrote :

Agreed, LGTM, please merge.

It might make sense in a followup commit to convert to using argparse's formatter_class=argparse.ArgumentDefaultsHelpFormatter instead of repeating the URL in the help message (it would also give notice that the default action for this script is to add a USN). Given that the script is usually inoked by wrapper scripts and not commonly invoked directly, it's not a particularly important thing.

Thanks!

review: Approve

« Back to merge proposal