)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8770,"name":"Ken Giusti","email":"kgiusti@gmail.com","username":"kgiusti"},"change_message_id":"4c0598c345c69f1a0f810b362164732da11561f0","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removed redundant \u0027the\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ie3fa917f595e8dbb747b410b2023a9ebd9d2f858"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a98d1f4_08572403","line":9,"updated":"2016-09-28 15:57:17.000000000","message":"You don\u0027t need to include \"TrivalFix\" - it doesn\u0027t describe what changes are made by this patch.\n\nIt is indeed straightforward - a single line commit message describing the change is enough.\n\nthanks.","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"},{"author":{"_account_id":19935,"name":"Anh Tran","email":"anhtt@vn.fujitsu.com","username":"trananhkma"},"change_message_id":"b9d25c0fb311cc685340dc71343afd566fdc7ff1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Removed redundant \u0027the\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"TrivialFix"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ie3fa917f595e8dbb747b410b2023a9ebd9d2f858"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a98d1f4_da540956","line":9,"in_reply_to":"3a98d1f4_08572403","updated":"2016-09-29 00:48:27.000000000","message":"Done","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"}],"oslo_messaging/_drivers/impl_rabbit.py":[{"author":{"_account_id":20864,"name":"Gevorg Davoian","email":"gdavoian@gmail.com","username":"gdavoian"},"change_message_id":"7fa79c89415dddd5010a818e2f5422ba946de315","unresolved":false,"context_lines":[{"line_number":374,"context_line":""},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"class ConnectionLock(DummyConnectionLock):"},{"line_number":377,"context_line":"    \"\"\"Lock object to protect access the kombu connection"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    This is a lock object to protect access the kombu connection"},{"line_number":380,"context_line":"    object between the heartbeat thread and the driver thread."}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_63399b41","line":377,"range":{"start_line":377,"start_character":7,"end_line":377,"end_character":11},"updated":"2016-09-28 07:52:58.000000000","message":"I don\u0027t see any reason to repeat the same phrase twice. I suggest merging this one and the next one into a single phrase.","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"},{"author":{"_account_id":19935,"name":"Anh Tran","email":"anhtt@vn.fujitsu.com","username":"trananhkma"},"change_message_id":"b9d25c0fb311cc685340dc71343afd566fdc7ff1","unresolved":false,"context_lines":[{"line_number":374,"context_line":""},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"class ConnectionLock(DummyConnectionLock):"},{"line_number":377,"context_line":"    \"\"\"Lock object to protect access the kombu connection"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    This is a lock object to protect access the kombu connection"},{"line_number":380,"context_line":"    object between the heartbeat thread and the driver thread."}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_ba5f3d71","line":377,"range":{"start_line":377,"start_character":7,"end_line":377,"end_character":11},"in_reply_to":"3a98d1f4_4d269e17","updated":"2016-09-29 00:48:27.000000000","message":"Done","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"},{"author":{"_account_id":20864,"name":"Gevorg Davoian","email":"gdavoian@gmail.com","username":"gdavoian"},"change_message_id":"54801d8997805be68750d704dc7a8b5cd1452800","unresolved":false,"context_lines":[{"line_number":374,"context_line":""},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"class ConnectionLock(DummyConnectionLock):"},{"line_number":377,"context_line":"    \"\"\"Lock object to protect access the kombu connection"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    This is a lock object to protect access the kombu connection"},{"line_number":380,"context_line":"    object between the heartbeat thread and the driver thread."}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_8dafa893","line":377,"range":{"start_line":377,"start_character":7,"end_line":377,"end_character":11},"in_reply_to":"5a8bc5a2_595e5077","updated":"2016-09-28 11:13:58.000000000","message":"Ok, got it, agree. But it seems that that the first \u0027the\u0027 has to be replaced with \u0027to\u0027, i.e. \u0027Lock object to protect access to the kombu connection...\u0027.","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"},{"author":{"_account_id":19935,"name":"Anh Tran","email":"anhtt@vn.fujitsu.com","username":"trananhkma"},"change_message_id":"47a39670319d06a71501c3211ca4b51692db8aa0","unresolved":false,"context_lines":[{"line_number":374,"context_line":""},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"class ConnectionLock(DummyConnectionLock):"},{"line_number":377,"context_line":"    \"\"\"Lock object to protect access the kombu connection"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    This is a lock object to protect access the kombu connection"},{"line_number":380,"context_line":"    object between the heartbeat thread and the driver thread."}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_595e5077","line":377,"range":{"start_line":377,"start_character":7,"end_line":377,"end_character":11},"in_reply_to":"5a8bc5a2_63399b41","updated":"2016-09-28 08:36:03.000000000","message":"I think the reason here:\n[H405] Multi line docstrings should start with a one line summary followed by an empty line.\nhttp://docs.openstack.org/developer/hacking/#docstrings","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"},{"author":{"_account_id":8770,"name":"Ken Giusti","email":"kgiusti@gmail.com","username":"kgiusti"},"change_message_id":"4c0598c345c69f1a0f810b362164732da11561f0","unresolved":false,"context_lines":[{"line_number":374,"context_line":""},{"line_number":375,"context_line":""},{"line_number":376,"context_line":"class ConnectionLock(DummyConnectionLock):"},{"line_number":377,"context_line":"    \"\"\"Lock object to protect access the kombu connection"},{"line_number":378,"context_line":""},{"line_number":379,"context_line":"    This is a lock object to protect access the kombu connection"},{"line_number":380,"context_line":"    object between the heartbeat thread and the driver thread."}],"source_content_type":"text/x-python","patch_set":1,"id":"3a98d1f4_4d269e17","line":377,"range":{"start_line":377,"start_character":7,"end_line":377,"end_character":11},"in_reply_to":"5a8bc5a2_8dafa893","updated":"2016-09-28 15:57:17.000000000","message":"+1 to adding the \"to\":\n\n\"Lock object to protect access to the kombu connection\"","commit_id":"24fcb38a5ca77308775c37420169434bdcfa422d"}]}
