)]}'
{"aodhclient/v2/alarm_history_cli.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"3c569e3a1717e2ee89cc6a7215e7926296fd6bb1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        query \u003d None"},{"line_number":34,"context_line":"        if parsed_args.query:"},{"line_number":35,"context_line":"            query \u003d jsonutils.dumps("},{"line_number":36,"context_line":"                utils.search_query_builder(parsed_args.query))"},{"line_number":37,"context_line":"        history \u003d self.app.client.alarm_history.search(query\u003dquery)"},{"line_number":38,"context_line":"        return utils.list2cols(self.COLS, history)"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa0719c6_5bf06c67","line":36,"range":{"start_line":36,"start_character":16,"end_line":36,"end_character":42},"updated":"2016-03-23 10:49:52.000000000","message":"Any reason not to use that as a type\u003d for the argument?","commit_id":"5e110e32f722771631feeade7de233e42452f6c1"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"288153057dd33cd798586665c659f29338d16bd0","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        query \u003d None"},{"line_number":34,"context_line":"        if parsed_args.query:"},{"line_number":35,"context_line":"            query \u003d jsonutils.dumps("},{"line_number":36,"context_line":"                utils.search_query_builder(parsed_args.query))"},{"line_number":37,"context_line":"        history \u003d self.app.client.alarm_history.search(query\u003dquery)"},{"line_number":38,"context_line":"        return utils.list2cols(self.COLS, history)"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa0719c6_4867c0b6","line":36,"range":{"start_line":36,"start_character":16,"end_line":36,"end_character":42},"in_reply_to":"fa0719c6_485d0014","updated":"2016-03-24 09:26:52.000000000","message":"Ah, yes, I think this idea is greate...","commit_id":"5e110e32f722771631feeade7de233e42452f6c1"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"ef7fb8aee29ef66a30c98c9766f756f9a863ff9c","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        query \u003d None"},{"line_number":34,"context_line":"        if parsed_args.query:"},{"line_number":35,"context_line":"            query \u003d jsonutils.dumps("},{"line_number":36,"context_line":"                utils.search_query_builder(parsed_args.query))"},{"line_number":37,"context_line":"        history \u003d self.app.client.alarm_history.search(query\u003dquery)"},{"line_number":38,"context_line":"        return utils.list2cols(self.COLS, history)"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa0719c6_66bc7ebb","line":36,"range":{"start_line":36,"start_character":16,"end_line":36,"end_character":42},"in_reply_to":"fa0719c6_5bf06c67","updated":"2016-03-24 04:08:45.000000000","message":"Sorry, I dont quite get you here.","commit_id":"5e110e32f722771631feeade7de233e42452f6c1"},{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"481fb66f7813543ffbb23d6bfed455cb11636a09","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        query \u003d None"},{"line_number":34,"context_line":"        if parsed_args.query:"},{"line_number":35,"context_line":"            query \u003d jsonutils.dumps("},{"line_number":36,"context_line":"                utils.search_query_builder(parsed_args.query))"},{"line_number":37,"context_line":"        history \u003d self.app.client.alarm_history.search(query\u003dquery)"},{"line_number":38,"context_line":"        return utils.list2cols(self.COLS, history)"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fa0719c6_485d0014","line":36,"range":{"start_line":36,"start_character":16,"end_line":36,"end_character":42},"in_reply_to":"fa0719c6_66bc7ebb","updated":"2016-03-24 09:24:03.000000000","message":"My suggestion was to set type\u003dutils.search_query_builder on your add_argument() on line 26. That make sure the parsing is done before going into take_action(), as argparse is going to use it directly. A good practice I\u0027d say.\n\nI hope I\u0027m clearer. :)","commit_id":"5e110e32f722771631feeade7de233e42452f6c1"}]}