)]}'
{"tools/get_action_list.py":[{"author":{"_account_id":5558,"name":"Winson Chan","email":"wcchan@stackstorm.com","username":"winson-c-chan"},"change_message_id":"7e5aaddee0dc7c9b1c58395ef78b6832b33816e4","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    parser.add_argument("},{"line_number":72,"context_line":"        \u0027service\u0027,"},{"line_number":73,"context_line":"        choices\u003d[\u0027nova\u0027, \u0027glance\u0027, \u0027heat\u0027, \u0027cinder\u0027, \u0027keystone\u0027, \u0027ceilometer\u0027,"},{"line_number":74,"context_line":"                 \u0027trove\u0027],"},{"line_number":75,"context_line":"        help\u003d\u0027Service name which methods need to be found.\u0027"},{"line_number":76,"context_line":"    )"},{"line_number":77,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_0947e6bc","line":74,"updated":"2015-12-02 18:30:58.000000000","message":"Since this list is growing longer, please change to the following style...\n\nchoices\u003d[\n    \u0027nova\u0027,\n    ...,\n    \u0027trove\u0027\n]\n\nSuggestion: maybe more manageable to define the list of choices as a constant and then assign the constant here?","commit_id":"84f04e6c3c4a214acc7c426b81aaec936ba6a285"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"7136ae8e952b18681114f5ebc49e4f78d00155f3","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    parser.add_argument("},{"line_number":72,"context_line":"        \u0027service\u0027,"},{"line_number":73,"context_line":"        choices\u003d[\u0027nova\u0027, \u0027glance\u0027, \u0027heat\u0027, \u0027cinder\u0027, \u0027keystone\u0027, \u0027ceilometer\u0027,"},{"line_number":74,"context_line":"                 \u0027trove\u0027],"},{"line_number":75,"context_line":"        help\u003d\u0027Service name which methods need to be found.\u0027"},{"line_number":76,"context_line":"    )"},{"line_number":77,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_13da9e4b","line":74,"in_reply_to":"9a8ffd7b_0947e6bc","updated":"2015-12-03 02:36:10.000000000","message":"Actually I think we can just use CLIENTS.keys()","commit_id":"84f04e6c3c4a214acc7c426b81aaec936ba6a285"},{"author":{"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},"change_message_id":"f24c1e737f8acd9db91efbc345cdcfb636c777c3","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    return cinderclient.Client()"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"def get_trove_client(**kwargs):"},{"line_number":140,"context_line":"    return troveclient.Client(\u0027\u0027, \u0027\u0027)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"CLIENTS \u003d {"},{"line_number":143,"context_line":"    \u0027nova\u0027: get_nova_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_d0de4c6c","line":140,"updated":"2015-12-03 01:59:02.000000000","message":"troveclient.Client(\u0027\u0027, \u0027\u0027) need two required arguments.","commit_id":"84f04e6c3c4a214acc7c426b81aaec936ba6a285"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"7136ae8e952b18681114f5ebc49e4f78d00155f3","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    return cinderclient.Client()"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"def get_trove_client(**kwargs):"},{"line_number":140,"context_line":"    return troveclient.Client(\u0027\u0027, \u0027\u0027)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"CLIENTS \u003d {"},{"line_number":143,"context_line":"    \u0027nova\u0027: get_nova_client,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_93c52e5d","line":140,"in_reply_to":"9a8ffd7b_d0de4c6c","updated":"2015-12-03 02:36:10.000000000","message":"This actually functionally works fine for generating output via this script. But I can add in some fake data to the strings here if you prefer.","commit_id":"84f04e6c3c4a214acc7c426b81aaec936ba6a285"}]}