)]}'
{"novaclient/exceptions.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"b896123f73458675a9ecfa2b2e0d1146ffa48375","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"class NoTokenLookupException(Exception):"},{"line_number":47,"context_line":"    \"\"\"Exception class for authentication with no endpoints looking up."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    This form of authentication does not support looking up"},{"line_number":50,"context_line":"    endpoints from an existing token."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_1cba57e2","line":47,"range":{"start_line":47,"start_character":50,"end_line":47,"end_character":70},"updated":"2015-12-03 19:08:47.000000000","message":"This part of the sentence doesn\u0027t make sense. How about something like:\n\n Exception class for authentication with no endpoint lookup.","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8859aad53ae731c1aa4b7e7d7fd3d65eeee41b6c","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"class NoTokenLookupException(Exception):"},{"line_number":47,"context_line":"    \"\"\"Exception class for authentication with no endpoints looking up."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    This form of authentication does not support looking up"},{"line_number":50,"context_line":"    endpoints from an existing token."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_073bc8d2","line":47,"in_reply_to":"7a740942_1cba57e2","updated":"2015-12-04 05:25:02.000000000","message":"ok, ll update.","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"}],"novaclient/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"5d4be60dd587be7a87ccbac6712c54a7a4bd3185","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"Bash completion."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_907f7b16","line":857,"updated":"2015-12-02 17:36:48.000000000","message":"This isn\u0027t going to work.  The help output is constructed from the first paragraph of the callback\u0027s docstring; what you\u0027ve done by adding this summary line like this is *decrease* the help available for the \"bash-completion\" subcommand.","commit_id":"e06555cf82c7de87c9c0f011e54c762ea4d5267c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f818b96923dc9ab631ad0105ca21cd294a916bbb","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"Bash completion."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_f3dd9f14","line":857,"in_reply_to":"9a8ffd7b_907f7b16","updated":"2015-12-03 00:14:11.000000000","message":"yea, right.\n\nDone","commit_id":"e06555cf82c7de87c9c0f011e54c762ea4d5267c"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"e9977a71892dab7ecf11f794f7bb42fa7d546afe","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"stdout all of the commands and options."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_58c55259","line":857,"updated":"2015-12-03 16:53:25.000000000","message":"This is still substantially less helpful than the original help text, and bodes ill for trying to fix the hacking check for the other do_* functions.  We need to output the descriptive paragraph.\n\nWe might be able to work around the problem by altering the code that generates the help text to skip a one-line summary if there\u0027s a following more descriptive paragraph; without that, I have to -1 changes like this…","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"e17e9db524cf37a22c741bf5d0b4355aaedb4934","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"stdout all of the commands and options."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_edcafea0","line":857,"in_reply_to":"7a740942_4726b069","updated":"2015-12-04 17:15:38.000000000","message":"The _code_ will still have the old descriptive paragraph.  The output of \"nova --help bash-completion\" will not, however, and that\u0027s the problem we have to resolve.","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"b896123f73458675a9ecfa2b2e0d1146ffa48375","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"stdout all of the commands and options."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_fc9d0b5a","line":857,"in_reply_to":"7a740942_58c55259","updated":"2015-12-03 19:08:47.000000000","message":"+1, I feel like we encountered the same problem last time we tried to enact H404 and H405 on the code\n\nhttps://review.openstack.org/#/c/164803/","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"53b9023a91f03445079b4dad5d244a005a0faa79","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"stdout all of the commands and options."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_72d47e1f","line":857,"in_reply_to":"7a740942_edcafea0","updated":"2015-12-07 08:39:52.000000000","message":"Thanks Kevin,\n\nBut i saw help also shows both-\n\n    x509-get-root-cert          Fetch the x509 root cert.\n    bash-completion             stdout all of the commands and options. Prints\n                                all of the commands and options to stdout so\n                                that the nova.bash_completion script doesn\u0027t\n                                have to hard code them.\n    help                        Display help about this program or one of its\n                                subcommands.","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8859aad53ae731c1aa4b7e7d7fd3d65eeee41b6c","unresolved":false,"context_lines":[{"line_number":854,"context_line":"            extension.run_hooks(hook_type, *args, **kwargs)"},{"line_number":855,"context_line":""},{"line_number":856,"context_line":"    def do_bash_completion(self, _args):"},{"line_number":857,"context_line":"        \"\"\"stdout all of the commands and options."},{"line_number":858,"context_line":""},{"line_number":859,"context_line":"        Prints all of the commands and options to stdout so that the"},{"line_number":860,"context_line":"        nova.bash_completion script doesn\u0027t have to hard code them."}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_4726b069","line":857,"in_reply_to":"7a740942_fc9d0b5a","updated":"2015-12-04 05:25:02.000000000","message":"yea I agree its not perfect summary line. But it will still have the old descriptive paragraph also right?","commit_id":"40ad5cb344ef195b6bfbebc3b5cd524b1fd0470c"}]}