)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jabar Ali \u003cjabbar.ali@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-10 19:47:55 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing version error from command nova keypair-add help"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If someone is using ‘nova keypair-add help’ instead of"},{"line_number":10,"context_line":"‘nova help keypair-add’ Then it is throwing error api"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_0dcf8a29","line":7,"updated":"2016-11-10 17:44:47.000000000","message":"this line needs to be \u003c50 chars per https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Removing version error from command nova keypair-add help"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If someone is using ‘nova keypair-add help’ instead of"},{"line_number":10,"context_line":"‘nova help keypair-add’ Then it is throwing error api"},{"line_number":11,"context_line":"error which is misleading and user is confused."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is happening because of the fact help word present"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_ad8e9ecd","line":10,"range":{"start_line":10,"start_character":24,"end_line":10,"end_character":25},"updated":"2016-11-10 17:44:47.000000000","message":"s/T/t/","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removing version error from command nova keypair-add help"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If someone is using ‘nova keypair-add help’ instead of"},{"line_number":10,"context_line":"‘nova help keypair-add’ Then it is throwing error api"},{"line_number":11,"context_line":"error which is misleading and user is confused."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is happening because of the fact help word present"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_adf77e57","line":10,"range":{"start_line":9,"start_character":44,"end_line":10,"end_character":23},"updated":"2016-11-10 17:44:47.000000000","message":"you\u0027re assuming they ran \u0027nova keypair-add help\u0027 by mistake. Instead, assume that was intentional, they really wanted to create a key named \"help\".","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If someone is using ‘nova keypair-add help’ instead of"},{"line_number":10,"context_line":"‘nova help keypair-add’ Then it is throwing error api"},{"line_number":11,"context_line":"error which is misleading and user is confused."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is happening because of the fact help word present"},{"line_number":14,"context_line":"in the argv is disabling The authentication and making"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_2da28e4b","line":11,"range":{"start_line":11,"start_character":15,"end_line":11,"end_character":46},"updated":"2016-11-10 17:44:47.000000000","message":"it\u0027s not just misleading, it\u0027s wrong. The command should have worked and created a key.","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":11,"context_line":"error which is misleading and user is confused."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This is happening because of the fact help word present"},{"line_number":14,"context_line":"in the argv is disabling The authentication and making"},{"line_number":15,"context_line":"wrong call flow of the function."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"To improve user experience ‘nova keypair-add help’"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_6d8806c6","line":14,"range":{"start_line":14,"start_character":25,"end_line":14,"end_character":26},"updated":"2016-11-10 17:44:47.000000000","message":"s/T/t/","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":14,"context_line":"in the argv is disabling The authentication and making"},{"line_number":15,"context_line":"wrong call flow of the function."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"To improve user experience ‘nova keypair-add help’"},{"line_number":18,"context_line":"will create keypair with name help. And nova help"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"keypair-add will display help of this command."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_cd64b223","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":26},"updated":"2016-11-10 17:44:47.000000000","message":"this isn\u0027t to improve user experience... it\u0027s to make the command do what it\u0027s supposed to do.","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"To improve user experience ‘nova keypair-add help’"},{"line_number":18,"context_line":"will create keypair with name help. And nova help"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"keypair-add will display help of this command."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes-Bug: #1559198"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_2d70cedb","line":19,"updated":"2016-11-10 17:44:47.000000000","message":"remove blank line in the middle of this sentence.","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":15,"context_line":"wrong call flow of the function."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"To improve user experience ‘nova keypair-add help’"},{"line_number":18,"context_line":"will create keypair with name help. And nova help"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"keypair-add will display help of this command."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes-Bug: #1559198"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_08077818","line":20,"range":{"start_line":18,"start_character":45,"end_line":20,"end_character":11},"updated":"2016-11-10 17:44:47.000000000","message":"surround with quotes","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"}],"novaclient/shell.py":[{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":621,"context_line":""},{"line_number":622,"context_line":"        self.setup_debugging(args.debug)"},{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_c82d8047","line":624,"range":{"start_line":624,"start_character":10,"end_line":624,"end_character":14},"updated":"2016-11-10 17:44:47.000000000","message":"I don\u0027t really understand what the TODO is here... shouldn\u0027t whatever we do here be the final and complete solution?","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":621,"context_line":""},{"line_number":622,"context_line":"        self.setup_debugging(args.debug)"},{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_28299c37","line":624,"range":{"start_line":624,"start_character":45,"end_line":624,"end_character":60},"updated":"2016-11-10 17:44:47.000000000","message":"redundant after \"how to handle\"","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":622,"context_line":"        self.setup_debugging(args.debug)"},{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "},{"line_number":628,"context_line":"        if \u0027keypair-add\u0027 \u003d\u003d argv[0]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_a8676cb5","line":625,"range":{"start_line":625,"start_character":39,"end_line":625,"end_character":40},"updated":"2016-11-10 17:44:47.000000000","message":"s/,/./","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":622,"context_line":"        self.setup_debugging(args.debug)"},{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "},{"line_number":628,"context_line":"        if \u0027keypair-add\u0027 \u003d\u003d argv[0]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_e80b24d4","line":625,"range":{"start_line":625,"start_character":54,"end_line":625,"end_character":61},"updated":"2016-11-10 17:44:47.000000000","message":"s/makying/making a/","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":622,"context_line":"        self.setup_debugging(args.debug)"},{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "},{"line_number":628,"context_line":"        if \u0027keypair-add\u0027 \u003d\u003d argv[0]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_a85dac16","line":625,"range":{"start_line":625,"start_character":18,"end_line":625,"end_character":39},"updated":"2016-11-10 17:44:47.000000000","message":"surround with quotes","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":623,"context_line":"        self.extensions \u003d []"},{"line_number":624,"context_line":"        # ToDo Need to evalute how to handle way of handling "},{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "},{"line_number":628,"context_line":"        if \u0027keypair-add\u0027 \u003d\u003d argv[0]:"},{"line_number":629,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and (\u0027help\u0027 !\u003d argv[1])) or ("}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_a8504cd5","line":626,"range":{"start_line":626,"start_character":10,"end_line":626,"end_character":14},"updated":"2016-11-10 17:44:47.000000000","message":"s/help,/named help./","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":625,"context_line":"        # Command nova keypair-add help, Currently is makying keypair"},{"line_number":626,"context_line":"        # help. and nova help keypair-add will display help "},{"line_number":627,"context_line":"        # of this command        "},{"line_number":628,"context_line":"        if \u0027keypair-add\u0027 \u003d\u003d argv[0]:"},{"line_number":629,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and (\u0027help\u0027 !\u003d argv[1])) or ("},{"line_number":630,"context_line":"               \u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":631,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_08dbf84f","line":628,"range":{"start_line":628,"start_character":25,"end_line":628,"end_character":35},"updated":"2016-11-10 17:44:47.000000000","message":"this won\u0027t work if you do something like \u0027nova --debug keypair-add\u0027.","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"ee09135cf7dc88db06bf35a3577bc70614a1f155","unresolved":false,"context_lines":[{"line_number":629,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and (\u0027help\u0027 !\u003d argv[1])) or ("},{"line_number":630,"context_line":"               \u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":631,"context_line":"        else:"},{"line_number":632,"context_line":"            do_help \u003d (\u0027help\u0027 in argv) or ("},{"line_number":633,"context_line":"               \u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":634,"context_line":"        # bash-completion should not require authentication"},{"line_number":635,"context_line":"        skip_auth \u003d do_help or ("}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_88d9c824","line":632,"updated":"2016-11-10 17:44:47.000000000","message":"I think you can drop the if/else and go back to a single do_help line, but replace (\u0027help\u0027 in argv) with (\u0027help\u0027 in argv and argv.index(\u0027help\u0027) \u003c argv.index(\u0027keypair-add\u0027)","commit_id":"9602c67a94fa273e98d184e73c513222ca89b502"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c474f0b7ea66b71075ce9822b926a9eec288186e","unresolved":false,"context_lines":[{"line_number":628,"context_line":"        # help as an argument as keypair name not help as help "},{"line_number":629,"context_line":"        if \u0027keypair-add\u0027 in argv:"},{"line_number":630,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":631,"context_line":"                        (argv.index(\u0027help\u0027) \u003c "},{"line_number":632,"context_line":"                        argv.index(\u0027keypair-add\u0027)))"},{"line_number":633,"context_line":"                        or (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) "},{"line_number":634,"context_line":"                        or not argv"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_1e086027","line":631,"updated":"2016-11-11 19:51:14.000000000","message":"your indentation is off for this and the following lines. You\u0027re lined up with a parenthesis block that you\u0027ve already closed.","commit_id":"42cb7cf5bd2a472236f53b9732b91a3791cd31ee"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c474f0b7ea66b71075ce9822b926a9eec288186e","unresolved":false,"context_lines":[{"line_number":630,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":631,"context_line":"                        (argv.index(\u0027help\u0027) \u003c "},{"line_number":632,"context_line":"                        argv.index(\u0027keypair-add\u0027)))"},{"line_number":633,"context_line":"                        or (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) "},{"line_number":634,"context_line":"                        or not argv"},{"line_number":635,"context_line":"        elif \u0027keypair-delete\u0027 in argv:"},{"line_number":636,"context_line":"                do_help \u003d ((\u0027help\u0027 in argv) and"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_5e3d78c2","line":633,"updated":"2016-11-11 19:51:14.000000000","message":"I think the \"or (\" is going to have to go on the previous line for python to understand that this is a continuation. Or you need to add another layer of parenthesis.","commit_id":"42cb7cf5bd2a472236f53b9732b91a3791cd31ee"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c474f0b7ea66b71075ce9822b926a9eec288186e","unresolved":false,"context_lines":[{"line_number":634,"context_line":"                        or not argv"},{"line_number":635,"context_line":"        elif \u0027keypair-delete\u0027 in argv:"},{"line_number":636,"context_line":"                do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":637,"context_line":"                            (argv.index(\u0027help\u0027) \u003c"},{"line_number":638,"context_line":"                            argv.index(\u0027keypair-delete\u0027))) or"},{"line_number":639,"context_line":"                            (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv)"},{"line_number":640,"context_line":"                            or not argv"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_be2e9489","line":637,"updated":"2016-11-11 19:51:14.000000000","message":"same issues as above","commit_id":"42cb7cf5bd2a472236f53b9732b91a3791cd31ee"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c474f0b7ea66b71075ce9822b926a9eec288186e","unresolved":false,"context_lines":[{"line_number":638,"context_line":"                            argv.index(\u0027keypair-delete\u0027))) or"},{"line_number":639,"context_line":"                            (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv)"},{"line_number":640,"context_line":"                            or not argv"},{"line_number":641,"context_line":"        else:"},{"line_number":642,"context_line":"            do_help \u003d (\u0027help\u0027 in argv) or ("},{"line_number":643,"context_line":"                       \u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) "},{"line_number":644,"context_line":"                       or not argv"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a629dbe_5eb2d84c","line":641,"updated":"2016-11-11 19:51:14.000000000","message":"I expect we have the same issue for keypair-show as for keypair-add and keypair-delete.","commit_id":"42cb7cf5bd2a472236f53b9732b91a3791cd31ee"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"f438fef71e24d22c14e80ed06e9c347cf8ee1619","unresolved":false,"context_lines":[{"line_number":628,"context_line":"        # help as an argument as keypair name not help as help"},{"line_number":629,"context_line":"        if \u0027keypair-add\u0027 in argv:"},{"line_number":630,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":631,"context_line":"                       (argv.index(\u0027help\u0027) \u003c argv.index(\u0027keypair-add\u0027))) or \\"},{"line_number":632,"context_line":"                (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":633,"context_line":"        elif \u0027keypair-delete\u0027 in argv:"},{"line_number":634,"context_line":"                do_help \u003d ((\u0027help\u0027 in argv) and"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_ce999970","line":631,"updated":"2016-11-15 16:41:53.000000000","message":"Please use an extra set of parentheses to eliminate the backslash continuation.","commit_id":"deb63606d59933b36e46ff9c3001b4b88d99cc0a"},{"author":{"_account_id":20477,"name":"Jabbar Ali","email":"jabbar.ali@gmail.com","username":"jabbar-ali"},"change_message_id":"0c62032afad4f52b917530cac66b5e163fd11a65","unresolved":false,"context_lines":[{"line_number":628,"context_line":"        # help as an argument as keypair name not help as help"},{"line_number":629,"context_line":"        if \u0027keypair-add\u0027 in argv:"},{"line_number":630,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":631,"context_line":"                       (argv.index(\u0027help\u0027) \u003c argv.index(\u0027keypair-add\u0027))) or \\"},{"line_number":632,"context_line":"                (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":633,"context_line":"        elif \u0027keypair-delete\u0027 in argv:"},{"line_number":634,"context_line":"                do_help \u003d ((\u0027help\u0027 in argv) and"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_fb2b60e9","line":631,"in_reply_to":"7a77a97e_ce999970","updated":"2016-11-16 05:29:32.000000000","message":"Just for the information is backslash is not permitted ?","commit_id":"deb63606d59933b36e46ff9c3001b4b88d99cc0a"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"14dd1fe3278a181b0d7e023c3404a3a097cd3cde","unresolved":false,"context_lines":[{"line_number":628,"context_line":"        # help as an argument as keypair name not help as help"},{"line_number":629,"context_line":"        if \u0027keypair-add\u0027 in argv:"},{"line_number":630,"context_line":"            do_help \u003d ((\u0027help\u0027 in argv) and"},{"line_number":631,"context_line":"                       (argv.index(\u0027help\u0027) \u003c argv.index(\u0027keypair-add\u0027))) or \\"},{"line_number":632,"context_line":"                (\u0027--help\u0027 in argv) or (\u0027-h\u0027 in argv) or not argv"},{"line_number":633,"context_line":"        elif \u0027keypair-delete\u0027 in argv:"},{"line_number":634,"context_line":"                do_help \u003d ((\u0027help\u0027 in argv) and"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a77a97e_2c4d806c","line":631,"in_reply_to":"7a77a97e_fb2b60e9","updated":"2016-11-16 15:58:29.000000000","message":"I\u0027m on a personal crusade against unnecessary backslash continuation.  I won\u0027t -1 for just a backslash, but I also won\u0027t +2 with a backslash unless it\u0027s a critical need.  For the record, the -1 I placed on this review was for the lack of regression tests.","commit_id":"deb63606d59933b36e46ff9c3001b4b88d99cc0a"}]}