)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"6c7b546a9e2e3788a39ea6a2c54ec2f457a3c815","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-09-30 11:12:21 +0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TrivialFix: Fix typo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I5ad121ec58ccc6e5f3cc499eca50d16e691f217e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a98d1f4_634c1443","line":8,"updated":"2016-09-29 13:00:32.000000000","message":"I am seeing too many patches with \u0027trivial fix\u0027 commits that don\u0027t say much. We let some merge but I am afraid that others think it is acceptable. Please add more information.","commit_id":"d2939abb9de7f726d54aefa90b364830943c6484"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"d9b2f96c30d5391a7273b9a2f2542f4740266cc3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nam Nguyen Hoai \u003cnamnh@vn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-14 06:22:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TrivialFix: Fix typo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_6aee83ed","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":20},"updated":"2016-11-09 04:18:43.000000000","message":"The first line would be better to be more specific such that:\n\n Fix two typos, \"messsage\" and \"containg\"","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":19554,"name":"Nam","email":"namnh68@fsoft.com.vn","username":"namnh"},"change_message_id":"03fdd132ca31504777e77c02bc5ce733f1f8a90a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nam Nguyen Hoai \u003cnamnh@vn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-14 06:22:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TrivialFix: Fix typo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a77a97e_71dfbdd5","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":20},"in_reply_to":"9a629dbe_6aee83ed","updated":"2016-11-21 01:54:12.000000000","message":"Thanks for your time. I updated these points. Could you please review it.","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"d9b2f96c30d5391a7273b9a2f2542f4740266cc3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TrivialFix: Fix typo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""},{"line_number":11,"context_line":"+ In utils.py file, it should be changed from \"containg\" to \"contains\""},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_ca7dcf6f","line":9,"range":{"start_line":9,"start_character":23,"end_line":9,"end_character":27},"updated":"2016-11-09 04:18:43.000000000","message":"s/some/two/","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":19554,"name":"Nam","email":"namnh68@fsoft.com.vn","username":"namnh"},"change_message_id":"03fdd132ca31504777e77c02bc5ce733f1f8a90a","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TrivialFix: Fix typo"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""},{"line_number":11,"context_line":"+ In utils.py file, it should be changed from \"containg\" to \"contains\""},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a77a97e_91f0d968","line":9,"range":{"start_line":9,"start_character":23,"end_line":9,"end_character":27},"in_reply_to":"9a629dbe_ca7dcf6f","updated":"2016-11-21 01:54:12.000000000","message":"Done","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"d9b2f96c30d5391a7273b9a2f2542f4740266cc3","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""},{"line_number":11,"context_line":"+ In utils.py file, it should be changed from \"containg\" to \"contains\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I5ad121ec58ccc6e5f3cc499eca50d16e691f217e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a629dbe_2aac8bec","line":11,"range":{"start_line":11,"start_character":61,"end_line":11,"end_character":69},"updated":"2016-11-09 04:18:43.000000000","message":"s/contains/containing/","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":19554,"name":"Nam","email":"namnh68@fsoft.com.vn","username":"namnh"},"change_message_id":"03fdd132ca31504777e77c02bc5ce733f1f8a90a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch set updated some wrong words:"},{"line_number":10,"context_line":"+ In error.py file, it should be changed from \"messsage\" to \"message\""},{"line_number":11,"context_line":"+ In utils.py file, it should be changed from \"containg\" to \"contains\""},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I5ad121ec58ccc6e5f3cc499eca50d16e691f217e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a77a97e_b1f31572","line":11,"range":{"start_line":11,"start_character":61,"end_line":11,"end_character":69},"in_reply_to":"9a629dbe_2aac8bec","updated":"2016-11-21 01:54:12.000000000","message":"Done","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"}],"ironic_python_agent/errors.py":[{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"400f2275fb03db5ddbbe78ff56d73bdefa260a5b","unresolved":false,"context_lines":[{"line_number":70,"context_line":"class InvalidCommandError(InvalidContentError):"},{"line_number":71,"context_line":"    \"\"\"Error which is raised when an unknown command is issued.\"\"\""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    message \u003d \u0027Invalid command\u0027"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def __init__(self, details):"},{"line_number":76,"context_line":"        super(InvalidCommandError, self).__init__(details)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_35db347e","line":73,"range":{"start_line":73,"start_character":3,"end_line":73,"end_character":11},"updated":"2016-09-29 07:06:56.000000000","message":"good catch!","commit_id":"d2939abb9de7f726d54aefa90b364830943c6484"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"b3f29ebcac219b4cc9000a43ed420f57e580b5ec","unresolved":false,"context_lines":[{"line_number":70,"context_line":"class InvalidCommandError(InvalidContentError):"},{"line_number":71,"context_line":"    \"\"\"Error which is raised when an unknown command is issued.\"\"\""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    message \u003d \u0027Invalid command\u0027"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def __init__(self, details):"},{"line_number":76,"context_line":"        super(InvalidCommandError, self).__init__(details)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_06f7628b","line":73,"updated":"2016-10-14 14:26:42.000000000","message":"Nice catch :)","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"a45a41be95238b994b453e14f789d641b6449ff8","unresolved":false,"context_lines":[{"line_number":70,"context_line":"class InvalidCommandError(InvalidContentError):"},{"line_number":71,"context_line":"    \"\"\"Error which is raised when an unknown command is issued.\"\"\""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    message \u003d \u0027Invalid command\u0027"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def __init__(self, details):"},{"line_number":76,"context_line":"        super(InvalidCommandError, self).__init__(details)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a77a97e_b13067ea","line":73,"updated":"2016-11-21 11:39:22.000000000","message":"great catch on this!","commit_id":"41cf45f126b4ebaac107b2602e658bbd27e0acc7"}],"ironic_python_agent/utils.py":[{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"b3f29ebcac219b4cc9000a43ed420f57e580b5ec","unresolved":false,"context_lines":[{"line_number":361,"context_line":"def gzip_and_b64encode(io_dict\u003dNone, file_list\u003dNone):"},{"line_number":362,"context_line":"    \"\"\"Gzip and base64 encode files and BytesIO buffers."},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    :param io_dict: A dictionary contains whose the keys are the file"},{"line_number":365,"context_line":"        names and the value a BytesIO object."},{"line_number":366,"context_line":"    :param file_list: A list of file path."},{"line_number":367,"context_line":"    :returns: A gzipped and base64 encoded string."}],"source_content_type":"text/x-python","patch_set":2,"id":"fa6399be_e6d42eed","line":364,"updated":"2016-10-14 14:26:42.000000000","message":"This English is incorrect.\n\n containing is the word to use.","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":19554,"name":"Nam","email":"namnh68@fsoft.com.vn","username":"namnh"},"change_message_id":"03fdd132ca31504777e77c02bc5ce733f1f8a90a","unresolved":false,"context_lines":[{"line_number":361,"context_line":"def gzip_and_b64encode(io_dict\u003dNone, file_list\u003dNone):"},{"line_number":362,"context_line":"    \"\"\"Gzip and base64 encode files and BytesIO buffers."},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    :param io_dict: A dictionary contains whose the keys are the file"},{"line_number":365,"context_line":"        names and the value a BytesIO object."},{"line_number":366,"context_line":"    :param file_list: A list of file path."},{"line_number":367,"context_line":"    :returns: A gzipped and base64 encoded string."}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_f1e90d80","line":364,"in_reply_to":"9a629dbe_eaa9b3e0","updated":"2016-11-21 01:54:12.000000000","message":"Done","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":13719,"name":"Naohiro Tamura","email":"naohirot.openstack@gmail.com","username":"nao"},"change_message_id":"d9b2f96c30d5391a7273b9a2f2542f4740266cc3","unresolved":false,"context_lines":[{"line_number":361,"context_line":"def gzip_and_b64encode(io_dict\u003dNone, file_list\u003dNone):"},{"line_number":362,"context_line":"    \"\"\"Gzip and base64 encode files and BytesIO buffers."},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    :param io_dict: A dictionary contains whose the keys are the file"},{"line_number":365,"context_line":"        names and the value a BytesIO object."},{"line_number":366,"context_line":"    :param file_list: A list of file path."},{"line_number":367,"context_line":"    :returns: A gzipped and base64 encoded string."}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_eaa9b3e0","line":364,"in_reply_to":"fa6399be_e6d42eed","updated":"2016-11-09 04:18:43.000000000","message":"+1","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"},{"author":{"_account_id":19554,"name":"Nam","email":"namnh68@fsoft.com.vn","username":"namnh"},"change_message_id":"03fdd132ca31504777e77c02bc5ce733f1f8a90a","unresolved":false,"context_lines":[{"line_number":361,"context_line":"def gzip_and_b64encode(io_dict\u003dNone, file_list\u003dNone):"},{"line_number":362,"context_line":"    \"\"\"Gzip and base64 encode files and BytesIO buffers."},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    :param io_dict: A dictionary contains whose the keys are the file"},{"line_number":365,"context_line":"        names and the value a BytesIO object."},{"line_number":366,"context_line":"    :param file_list: A list of file path."},{"line_number":367,"context_line":"    :returns: A gzipped and base64 encoded string."}],"source_content_type":"text/x-python","patch_set":2,"id":"7a77a97e_3100e533","line":364,"in_reply_to":"fa6399be_e6d42eed","updated":"2016-11-21 01:54:12.000000000","message":"Thanks for your comment. I have just updated this point in next patch set. :)","commit_id":"2d0a3763ec7efba6f392d272d1a03ac1fb116dd2"}]}