)]}'
{"git_review/cmd.py":[{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"f6ca07ab6845a5eece81e2ba72c043904b8050a3","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"                        action\u003d\"store_true\","},{"line_number":1437,"context_line":"                        help\u003d\"Use remote push-url logic instead of separate\""},{"line_number":1438,"context_line":"                             \" remotes\")"},{"line_number":1439,"context_line":"    parser.add_argument(\"--notify\", dest\u003d\"notify\","},{"line_number":1440,"context_line":"                        help\u003d\"Control to whom email notifications are sent,\""},{"line_number":1441,"context_line":"                             \" e.g. OWNER\")"},{"line_number":1442,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"3fa7e38b_23dfb267","line":1439,"range":{"start_line":1439,"start_character":24,"end_line":1439,"end_character":34},"updated":"2020-01-27 19:29:14.000000000","message":"you did not include the -n short form here. missed?","commit_id":"3c4910479608f5fafc8fb69bb77b97c36b993844"},{"author":{"_account_id":6415,"name":"Miklos Vajna","email":"vmiklos@collabora.com","username":"vmiklos"},"change_message_id":"fa58abf81c8392a41b04a2c594cabd6a8e14e76b","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"                        action\u003d\"store_true\","},{"line_number":1437,"context_line":"                        help\u003d\"Use remote push-url logic instead of separate\""},{"line_number":1438,"context_line":"                             \" remotes\")"},{"line_number":1439,"context_line":"    parser.add_argument(\"--notify\", dest\u003d\"notify\","},{"line_number":1440,"context_line":"                        help\u003d\"Control to whom email notifications are sent,\""},{"line_number":1441,"context_line":"                             \" e.g. OWNER\")"},{"line_number":1442,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"3fa7e38b_ba689cbb","line":1439,"range":{"start_line":1439,"start_character":24,"end_line":1439,"end_character":34},"in_reply_to":"3fa7e38b_23dfb267","updated":"2020-01-28 09:27:25.000000000","message":"-n is already an alias for --dry-run, so I propose to not have a short form for this new switch.","commit_id":"3c4910479608f5fafc8fb69bb77b97c36b993844"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"f6ca07ab6845a5eece81e2ba72c043904b8050a3","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"                        action\u003d\"store_true\","},{"line_number":1437,"context_line":"                        help\u003d\"Use remote push-url logic instead of separate\""},{"line_number":1438,"context_line":"                             \" remotes\")"},{"line_number":1439,"context_line":"    parser.add_argument(\"--notify\", dest\u003d\"notify\","},{"line_number":1440,"context_line":"                        help\u003d\"Control to whom email notifications are sent,\""},{"line_number":1441,"context_line":"                             \" e.g. OWNER\")"},{"line_number":1442,"context_line":""},{"line_number":1443,"context_line":"    rebase_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":1444,"context_line":"    rebase_group.add_argument(\"-R\", \"--no-rebase\", dest\u003d\"rebase\","}],"source_content_type":"text/x-python","patch_set":11,"id":"3fa7e38b_c3eede1e","line":1441,"range":{"start_line":1439,"start_character":24,"end_line":1441,"end_character":43},"updated":"2020-01-27 19:29:14.000000000","message":"Be sure you specify list of acceptable values NONE, OWNER, OWNER_REVIEWERS, ALL\n\nUse https://docs.python.org/3/library/argparse.html#choices for this.\n\nAnd be sure that when I run \"--help\" it will display accepted values.","commit_id":"3c4910479608f5fafc8fb69bb77b97c36b993844"},{"author":{"_account_id":6415,"name":"Miklos Vajna","email":"vmiklos@collabora.com","username":"vmiklos"},"change_message_id":"fa58abf81c8392a41b04a2c594cabd6a8e14e76b","unresolved":false,"context_lines":[{"line_number":1436,"context_line":"                        action\u003d\"store_true\","},{"line_number":1437,"context_line":"                        help\u003d\"Use remote push-url logic instead of separate\""},{"line_number":1438,"context_line":"                             \" remotes\")"},{"line_number":1439,"context_line":"    parser.add_argument(\"--notify\", dest\u003d\"notify\","},{"line_number":1440,"context_line":"                        help\u003d\"Control to whom email notifications are sent,\""},{"line_number":1441,"context_line":"                             \" e.g. OWNER\")"},{"line_number":1442,"context_line":""},{"line_number":1443,"context_line":"    rebase_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":1444,"context_line":"    rebase_group.add_argument(\"-R\", \"--no-rebase\", dest\u003d\"rebase\","}],"source_content_type":"text/x-python","patch_set":11,"id":"3fa7e38b_3a5cac18","line":1441,"range":{"start_line":1439,"start_character":24,"end_line":1441,"end_character":43},"in_reply_to":"3fa7e38b_c3eede1e","updated":"2020-01-28 09:27:25.000000000","message":"Done. --help opens the man page, but I verified that -h indeed shows this (out of the box, thanks argparse).","commit_id":"3c4910479608f5fafc8fb69bb77b97c36b993844"}]}