)]}'
{"cliff/help.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"8b69b84b17107949806b18ab136082d6bda06685","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import sys"},{"line_number":16,"context_line":"import traceback"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from . import command"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class HelpAction(argparse.Action):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf091321_df7c1db7","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":21},"updated":"2017-06-08 09:58:37.000000000","message":"At some point, you might change this to use the actual module name. I was pretty sure that was \"The OpenStack Way\"","commit_id":"804b384e90ae2d1bb1e1517cc8085d97d38dd58f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4f31f8c65cb270a6a2a3a606b7123599a269c80c","unresolved":false,"context_lines":[{"line_number":39,"context_line":"                    traceback.print_exc(file\u003dapp.stdout)"},{"line_number":40,"context_line":"                continue"},{"line_number":41,"context_line":"            try:"},{"line_number":42,"context_line":"                kwargs \u003d {}"},{"line_number":43,"context_line":"                if \u0027cmd_name\u0027 in inspect.getargspec(factory.__init__).args:"},{"line_number":44,"context_line":"                    kwargs[\u0027cmd_name\u0027] \u003d name"},{"line_number":45,"context_line":"                cmd \u003d factory(app, None, **kwargs)"},{"line_number":46,"context_line":"                if cmd.deprecated:"},{"line_number":47,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_983d952e","line":44,"range":{"start_line":42,"start_character":0,"end_line":44,"end_character":45},"updated":"2017-06-07 12:34:13.000000000","message":"nit: does all of this need to be inside the try?","commit_id":"804b384e90ae2d1bb1e1517cc8085d97d38dd58f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"7382f811e2be2fe6850108cd056c67e1ae7d4ac8","unresolved":false,"context_lines":[{"line_number":39,"context_line":"                    traceback.print_exc(file\u003dapp.stdout)"},{"line_number":40,"context_line":"                continue"},{"line_number":41,"context_line":"            try:"},{"line_number":42,"context_line":"                kwargs \u003d {}"},{"line_number":43,"context_line":"                if \u0027cmd_name\u0027 in inspect.getargspec(factory.__init__).args:"},{"line_number":44,"context_line":"                    kwargs[\u0027cmd_name\u0027] \u003d name"},{"line_number":45,"context_line":"                cmd \u003d factory(app, None, **kwargs)"},{"line_number":46,"context_line":"                if cmd.deprecated:"},{"line_number":47,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_a125bdad","line":44,"range":{"start_line":42,"start_character":0,"end_line":44,"end_character":45},"in_reply_to":"df140735_983d952e","updated":"2017-06-07 15:47:59.000000000","message":"If a specified factor is not a class, factory.__init__ may raise an exception, though it is completely wrong usage..","commit_id":"804b384e90ae2d1bb1e1517cc8085d97d38dd58f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"72d575581388b02b4597214e809d3fc615299475","unresolved":false,"context_lines":[{"line_number":39,"context_line":"                    traceback.print_exc(file\u003dapp.stdout)"},{"line_number":40,"context_line":"                continue"},{"line_number":41,"context_line":"            try:"},{"line_number":42,"context_line":"                kwargs \u003d {}"},{"line_number":43,"context_line":"                if \u0027cmd_name\u0027 in inspect.getargspec(factory.__init__).args:"},{"line_number":44,"context_line":"                    kwargs[\u0027cmd_name\u0027] \u003d name"},{"line_number":45,"context_line":"                cmd \u003d factory(app, None, **kwargs)"},{"line_number":46,"context_line":"                if cmd.deprecated:"},{"line_number":47,"context_line":"                    continue"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_8d0c2843","line":44,"range":{"start_line":42,"start_character":0,"end_line":44,"end_character":45},"in_reply_to":"df140735_a125bdad","updated":"2017-06-07 17:48:39.000000000","message":"Yeah, I wasn\u0027t anticipating any new error cases. I just put the kwargs stuff near where it was being used.","commit_id":"804b384e90ae2d1bb1e1517cc8085d97d38dd58f"}]}