)]}'
{"pegleg/cli.py":[{"author":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"change_message_id":"3a63dfc8f57f7c4ef237136e027b97345c9510f6","unresolved":false,"context_lines":[{"line_number":588,"context_line":"    expired_certs_exist, cert_results \u003d engine.secrets.check_cert_expiry("},{"line_number":589,"context_line":"        site_name, duration\u003ddays)"},{"line_number":590,"context_line":""},{"line_number":591,"context_line":"    if not expired_certs_exist:"},{"line_number":592,"context_line":"        click.echo(\"No certificates will expire within {} days.\".format(days))"},{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_452a087e","line":591,"range":{"start_line":591,"start_character":4,"end_line":591,"end_character":31},"updated":"2019-12-10 15:29:35.000000000","message":"Any reason to use not instead of flipping the if/else to avoid the not? Just a style preference, but logic seems easier if you can avoid a not.","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"},{"author":{"_account_id":30173,"name":"Ian Pittwood","email":"pittwoodian@gmail.com","username":"ianp"},"change_message_id":"40bdb87297fab293a79e65ec26a606271d0bbeea","unresolved":false,"context_lines":[{"line_number":588,"context_line":"    expired_certs_exist, cert_results \u003d engine.secrets.check_cert_expiry("},{"line_number":589,"context_line":"        site_name, duration\u003ddays)"},{"line_number":590,"context_line":""},{"line_number":591,"context_line":"    if not expired_certs_exist:"},{"line_number":592,"context_line":"        click.echo(\"No certificates will expire within {} days.\".format(days))"},{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_25610c3a","line":591,"range":{"start_line":591,"start_character":4,"end_line":591,"end_character":31},"in_reply_to":"3fa7e38b_452a087e","updated":"2019-12-10 16:02:12.000000000","message":"Done","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"},{"author":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"change_message_id":"3a63dfc8f57f7c4ef237136e027b97345c9510f6","unresolved":false,"context_lines":[{"line_number":589,"context_line":"        site_name, duration\u003ddays)"},{"line_number":590,"context_line":""},{"line_number":591,"context_line":"    if not expired_certs_exist:"},{"line_number":592,"context_line":"        click.echo(\"No certificates will expire within {} days.\".format(days))"},{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"},{"line_number":595,"context_line":"        click.echo("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_4501a803","line":592,"range":{"start_line":592,"start_character":48,"end_line":592,"end_character":63},"updated":"2019-12-10 15:29:35.000000000","message":"within the next {} days.? Not sure if it really adds clarity?","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"},{"author":{"_account_id":30173,"name":"Ian Pittwood","email":"pittwoodian@gmail.com","username":"ianp"},"change_message_id":"40bdb87297fab293a79e65ec26a606271d0bbeea","unresolved":false,"context_lines":[{"line_number":589,"context_line":"        site_name, duration\u003ddays)"},{"line_number":590,"context_line":""},{"line_number":591,"context_line":"    if not expired_certs_exist:"},{"line_number":592,"context_line":"        click.echo(\"No certificates will expire within {} days.\".format(days))"},{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"},{"line_number":595,"context_line":"        click.echo("}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_45644849","line":592,"range":{"start_line":592,"start_character":48,"end_line":592,"end_character":63},"in_reply_to":"3fa7e38b_4501a803","updated":"2019-12-10 16:02:12.000000000","message":"Done","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"},{"author":{"_account_id":28235,"name":"Aaron Sheffield","email":"ajs@sheffieldfamily.net","username":"aaronsheffield"},"change_message_id":"3a63dfc8f57f7c4ef237136e027b97345c9510f6","unresolved":false,"context_lines":[{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"},{"line_number":595,"context_line":"        click.echo("},{"line_number":596,"context_line":"            \"The following certs will expire within {} days: \\n{}\".format("},{"line_number":597,"context_line":"                days, cert_results))"},{"line_number":598,"context_line":"        exit(1)"},{"line_number":599,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_c50cb83a","line":596,"range":{"start_line":596,"start_character":45,"end_line":596,"end_character":59},"updated":"2019-12-10 15:29:35.000000000","message":"Ditto.","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"},{"author":{"_account_id":30173,"name":"Ian Pittwood","email":"pittwoodian@gmail.com","username":"ianp"},"change_message_id":"40bdb87297fab293a79e65ec26a606271d0bbeea","unresolved":false,"context_lines":[{"line_number":593,"context_line":"        exit(0)"},{"line_number":594,"context_line":"    else:"},{"line_number":595,"context_line":"        click.echo("},{"line_number":596,"context_line":"            \"The following certs will expire within {} days: \\n{}\".format("},{"line_number":597,"context_line":"                days, cert_results))"},{"line_number":598,"context_line":"        exit(1)"},{"line_number":599,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_e556945e","line":596,"range":{"start_line":596,"start_character":45,"end_line":596,"end_character":59},"in_reply_to":"3fa7e38b_c50cb83a","updated":"2019-12-10 16:02:12.000000000","message":"Done","commit_id":"2c69523739c5afb336323a7fb2b820f864f247aa"}]}