Code review comment for ~rodrigo-zaiden/ubuntu-cve-tracker:check_cves_uris

Revision history for this message
Mark Esler (eslerm) wrote :

Thank you Rodrigo!

Small style nitpick is that `if not args.uris:` feels cleaner than `if len(args.uris) == 0:`

Since an empty list evaluates to False: `bool([])`

(note that `nargs='*'` implicitly sets args.uris to a list)

review: Approve

« Back to merge proposal