Looks good! Just a couple of small things:
16 + except httplib2.SSLHandshakeError: 17 + raise CommandError( 18 + "Certificate verify failed, use --disable-cert-check to disable " 19 + "the certificate check.")
s/verify/verification/
37 + parser.add_argument( 38 + '--disable-cert-check', action='store_true', help=( 39 + "Disable SSL certificate check"), default=False)
This is a long option and will get a bit annoying if someone needs to repeatedly specify it (e.g. on a dev system), can you make a short one too please, e.g. -d
Thanks!
« Back to merge proposal
Looks good! Just a couple of small things:
16 + except httplib2. SSLHandshakeErr or: cert-check to disable "
17 + raise CommandError(
18 + "Certificate verify failed, use --disable-
19 + "the certificate check.")
s/verify/ verification/
37 + parser. add_argument( cert-check' , action= 'store_ true', help=(
38 + '--disable-
39 + "Disable SSL certificate check"), default=False)
This is a long option and will get a bit annoying if someone needs to repeatedly specify it (e.g. on a dev system), can you make a short one too please, e.g. -d
Thanks!