)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"15ec4d5f77533d952c4c3d1520e9099eaba3cc8d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Modify some codes in order"},{"line_number":10,"context_line":"to make them meet the Python3"},{"line_number":11,"context_line":"common pattern."},{"line_number":12,"context_line":"In this patch,I replace"},{"line_number":13,"context_line":"filter(lambda obj: test(obj),data)"},{"line_number":14,"context_line":"with[obj for obj in data if test(obj)]"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba5201f7_073928d7","line":11,"updated":"2017-01-06 08:56:38.000000000","message":"This should be wrapped at 72 characters, not 29 :)","commit_id":"7b6a0be2633903748563fe0233d529254a2885e2"}],"mistral/engine/dispatcher.py":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"15ec4d5f77533d952c4c3d1520e9099eaba3cc8d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    \"\"\""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    # Remove all \u0027noop\u0027 commands."},{"line_number":52,"context_line":"    cmds \u003d list([c for c in cmds if not isinstance(c, commands.Noop)])"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    state_cmd_idx \u003d -1"},{"line_number":55,"context_line":"    state_cmd \u003d None"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5201f7_6750340c","line":52,"updated":"2017-01-06 08:56:38.000000000","message":"I believe [ and ] are not required when passing into a list - but it is fine to leave them in. This comment is true for many of the other changes too.","commit_id":"7b6a0be2633903748563fe0233d529254a2885e2"}],"mistral/engine/tasks.py":[{"author":{"_account_id":9712,"name":"Dougal Matthews","email":"dougal.matthews@canonical.com","username":"dougal"},"change_message_id":"fac079259cca137554cd6984cef7594bcf5b1235","unresolved":false,"context_lines":[{"line_number":327,"context_line":"        if self.reset_flag:"},{"line_number":328,"context_line":"            execs \u003d self.task_ex.executions"},{"line_number":329,"context_line":"        else:"},{"line_number":330,"context_line":"            execs \u003d [e for e in self.task_ex_executions if"},{"line_number":331,"context_line":"                     (e.accepted and"},{"line_number":332,"context_line":"                      e.state in [states.ERROR, states.CANCELLED])]"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba5201f7_d53d03de","line":330,"updated":"2017-01-06 12:22:57.000000000","message":"This should be self.task_ex.executions","commit_id":"7b6a0be2633903748563fe0233d529254a2885e2"}]}