)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"1acebf99bc89baf357ec032768dba238aaf08705","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Clean the redundant code in shell.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the file cinderclient/shell.py, packages requests and six"},{"line_number":10,"context_line":"are double imported, This patch is to clean the redundant oode."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bade57753a50245df1b4b82c4a8708d3b0acdba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff346bd7_24c70a99","line":10,"range":{"start_line":10,"start_character":19,"end_line":10,"end_character":20},"updated":"2017-07-27 07:55:32.000000000","message":". instead of comma","commit_id":"3206fd11c9bd63704a6ed3a09a91cd5010d3c906"},{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"1acebf99bc89baf357ec032768dba238aaf08705","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Clean the redundant code in shell.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the file cinderclient/shell.py, packages requests and six"},{"line_number":10,"context_line":"are double imported, This patch is to clean the redundant oode."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bade57753a50245df1b4b82c4a8708d3b0acdba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ff346bd7_04964694","line":10,"range":{"start_line":10,"start_character":58,"end_line":10,"end_character":59},"updated":"2017-07-27 07:55:32.000000000","message":"nit: code","commit_id":"3206fd11c9bd63704a6ed3a09a91cd5010d3c906"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"2cd81855b40375cc2f3018b59c11c4fb0fe46089","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In the file cinderclient/shell.py, packages requests and six"},{"line_number":10,"context_line":"are double imported. This patch is to clean the redundant code."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4bade57753a50245df1b4b82c4a8708d3b0acdba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"ff346bd7_4fdb3b7b","line":11,"updated":"2017-07-27 15:13:09.000000000","message":"nit: please add TrivialFix flag if you do one more patch","commit_id":"c6504502edcda34c624af4f3c5e1765c664f408e"}],"cinderclient/shell.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"99fe1d988a7d680a2b9da76caec8550143e4b7c3","unresolved":false,"context_lines":[{"line_number":24,"context_line":"import logging"},{"line_number":25,"context_line":"import sys"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from keystoneauth1 import discover"},{"line_number":31,"context_line":"from keystoneauth1 import loading"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff346bd7_c3b50dcf","line":28,"range":{"start_line":27,"start_character":3,"end_line":28,"end_character":10},"updated":"2017-07-25 23:25:20.000000000","message":"While you\u0027re at it, these should not be a separate group. Please add below osprofiler and above six.moves without spacing to properly group and order third party imports.","commit_id":"f1bf739ebdce444dd3f60ac1e9e4f05d765ab71a"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"147d3a5e9e3e6050e2c9dc568a26cc0fac8c72cc","unresolved":false,"context_lines":[{"line_number":24,"context_line":"import logging"},{"line_number":25,"context_line":"import sys"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":"import six.moves.urllib.parse as urlparse"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from keystoneauth1 import discover"},{"line_number":32,"context_line":"from keystoneauth1.exceptions import DiscoveryFailure"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff346bd7_fc8ad967","line":29,"range":{"start_line":27,"start_character":0,"end_line":29,"end_character":41},"updated":"2017-07-26 15:53:47.000000000","message":"This should all be under the keystoneauth1 and oslo_util imports, grouped together.","commit_id":"3a7af1e5c35e64a4fb8cf2596aac35dfdb138dd7"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"147d3a5e9e3e6050e2c9dc568a26cc0fac8c72cc","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"import cinderclient"},{"line_number":42,"context_line":"from cinderclient import _i18n"},{"line_number":43,"context_line":"# Enable i18n lazy translation"},{"line_number":44,"context_line":"_i18n.enable_lazy()"},{"line_number":45,"context_line":"from cinderclient._i18n import _"},{"line_number":46,"context_line":"from cinderclient import api_versions"},{"line_number":47,"context_line":"from cinderclient import client"}],"source_content_type":"text/x-python","patch_set":3,"id":"ff346bd7_5c8fa574","line":44,"range":{"start_line":43,"start_character":0,"end_line":44,"end_character":19},"updated":"2017-07-26 15:53:47.000000000","message":"This should have remained after all the imports.","commit_id":"3a7af1e5c35e64a4fb8cf2596aac35dfdb138dd7"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"0b3d48cf677d7c8bf2bc2d30e0991e8f3cddfbe1","unresolved":false,"context_lines":[{"line_number":25,"context_line":"import sys"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from keystoneauth1 import discover"},{"line_number":28,"context_line":"from keystoneauth1.exceptions import DiscoveryFailure"},{"line_number":29,"context_line":"from keystoneauth1.identity import v2 as v2_auth"},{"line_number":30,"context_line":"from keystoneauth1.identity import v3 as v3_auth"},{"line_number":31,"context_line":"from keystoneauth1 import loading"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff346bd7_96339ef6","line":28,"range":{"start_line":28,"start_character":37,"end_line":28,"end_character":53},"updated":"2017-07-26 21:26:13.000000000","message":"Shouldn\u0027t be importing classes directly, but I see this is existing code.","commit_id":"3206fd11c9bd63704a6ed3a09a91cd5010d3c906"}]}