)]}'
{"mistral/api/controllers/v2/execution.py":[{"author":{"_account_id":18955,"name":"dharmendra","email":"dharmendra.kushwaha@gmail.com","username":"dharmendra"},"change_message_id":"9c49b15f0d0117b18dcc4024cd4d2b4591761e72","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from mistral.utils import filter_utils"},{"line_number":36,"context_line":"from mistral.utils import merge_dicts"},{"line_number":37,"context_line":"from mistral.utils import rest_utils"},{"line_number":38,"context_line":"from mistral.utils.rest_utils import fields_list_to_cls_fields_tuple"},{"line_number":39,"context_line":"from mistral.workflow import states"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_1929e0ee","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":68},"updated":"2018-12-07 06:33:11.000000000","message":"why to import again, as rest_utils is already imported in above line.And in line #74, it can be called it as rest_utils.fields_list_to_cls_fields_tuple","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c762a2ac50443c09f6b9d1ee6ddc47e1dd13fcea","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from mistral.utils import filter_utils"},{"line_number":36,"context_line":"from mistral.utils import merge_dicts"},{"line_number":37,"context_line":"from mistral.utils import rest_utils"},{"line_number":38,"context_line":"from mistral.utils.rest_utils import fields_list_to_cls_fields_tuple"},{"line_number":39,"context_line":"from mistral.workflow import states"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"LOG \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfd5e7cf_4922a9a6","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":68},"in_reply_to":"3f79a3b5_1929e0ee","updated":"2019-01-09 09:20:02.000000000","message":"Yep, weird. I agree.","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c762a2ac50443c09f6b9d1ee6ddc47e1dd13fcea","unresolved":false,"context_lines":[{"line_number":96,"context_line":"        :param id: UUID of execution to retrieve."},{"line_number":97,"context_line":"        :param fields: Optional. A specified list of fields of the resource to"},{"line_number":98,"context_line":"                       be returned. \u0027id\u0027 will be included automatically in"},{"line_number":99,"context_line":"                       fields if it\u0027s provided, since it will be used when"},{"line_number":100,"context_line":"                       constructing \u0027next\u0027 link."},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"dfd5e7cf_e9459d44","line":99,"range":{"start_line":99,"start_character":30,"end_line":99,"end_character":46},"updated":"2019-01-09 09:20:02.000000000","message":"I guess \"if it\u0027s not provided\"?","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"}],"mistral/tests/unit/api/v2/test_executions.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c762a2ac50443c09f6b9d1ee6ddc47e1dd13fcea","unresolved":false,"context_lines":[{"line_number":171,"context_line":"        self.assertEqual(200, resp.status_int)"},{"line_number":172,"context_line":"        self.assertEqual(1, len(resp.json))"},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"        expected_json \u003d {"},{"line_number":175,"context_line":"            \u0027state\u0027: \u0027RUNNING\u0027"},{"line_number":176,"context_line":"        }"},{"line_number":177,"context_line":"        self.assertDictEqual(expected_json, resp.json)"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"    @mock.patch.object(db_api, \u0027get_workflow_execution\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfd5e7cf_69a8cdda","line":176,"range":{"start_line":174,"start_character":0,"end_line":176,"end_character":9},"updated":"2019-01-09 09:20:02.000000000","message":"Soft/style: could be defined one line.","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"}],"mistral/utils/rest_utils.py":[{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c762a2ac50443c09f6b9d1ee6ddc47e1dd13fcea","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        )"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"def fields_list_to_cls_fields_tuple(fields_list, cls):"},{"line_number":123,"context_line":"    if not fields_list:"},{"line_number":124,"context_line":"        return ()"},{"line_number":125,"context_line":"    return tuple([getattr(cls, str(field)) for field in fields_list])"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfd5e7cf_29d8857d","line":122,"range":{"start_line":122,"start_character":4,"end_line":122,"end_character":15},"updated":"2019-01-09 09:20:02.000000000","message":"\u003d\u003e field_list","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"},{"author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"change_message_id":"c762a2ac50443c09f6b9d1ee6ddc47e1dd13fcea","unresolved":false,"context_lines":[{"line_number":119,"context_line":"        )"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"def fields_list_to_cls_fields_tuple(fields_list, cls):"},{"line_number":123,"context_line":"    if not fields_list:"},{"line_number":124,"context_line":"        return ()"},{"line_number":125,"context_line":"    return tuple([getattr(cls, str(field)) for field in fields_list])"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfd5e7cf_a9ec7529","line":122,"range":{"start_line":122,"start_character":36,"end_line":122,"end_character":47},"updated":"2019-01-09 09:20:02.000000000","message":"Soft/style: I\u0027d call it \"field_list\" since \"list\" already points to plural (a little better English)","commit_id":"c821a4475fd7c27928a81be87d0203915a5e2337"}]}