)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7575,"name":"Sabari","email":"smurugesan@vmware.com","username":"sabari"},"change_message_id":"93c03d60971f4c0b00225d8f58e0b0008d63386e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     kairat_kushaev \u003ckkushaev@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-12-03 18:42:11 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Import i18 functions directly"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Import i18n functions from module directly and do not use"},{"line_number":10,"context_line":"global module variables like _ \u003d i18n._. It makes code shorter"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a740942_28eea9b4","line":7,"updated":"2015-12-04 23:22:04.000000000","message":"i18n - but nevermind :)","commit_id":"48920010a209d20d02a21dce3c66122d51f0c38a"}],"glance/cmd/__init__.py":[{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"e056db6d2fa8048b7219e1d0d98fa2e5cc523705","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import oslo_utils.strutils as strutils"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from glance import i18n"},{"line_number":22,"context_line":"from glance.i18n import _"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"try:"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a740942_5997143a","line":21,"updated":"2015-12-07 15:12:52.000000000","message":"I think this is redundant.","commit_id":"de7d898bb52338aba48028f481ffa519ca5c8adc"}],"glance/common/auth.py":[{"author":{"_account_id":17976,"name":"Drew Varner","email":"avarner@linux.vnet.ibm.com","username":"avarner"},"change_message_id":"a5dc192cb4d09e35966d98de54fe59841b082025","unresolved":false,"context_lines":[{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        self.check_auth_params()"},{"line_number":127,"context_line":"        auth_url \u003d self.creds[\u0027auth_url\u0027]"},{"line_number":128,"context_line":"        for i in range(self.MAX_REDIRECTS):"},{"line_number":129,"context_line":"            try:"},{"line_number":130,"context_line":"                _authenticate(auth_url)"},{"line_number":131,"context_line":"            except exception.AuthorizationRedirect as e:"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a8ffd7b_7727a464","line":128,"updated":"2015-11-25 19:52:05.000000000","message":"PEP8 (the document, not the automated test) says no single character variables - I\u0027d suggest \"dummy\", \"ignore\", or \"unused\". It\u0027s not a major issue, and I\u0027d probably approve \"i\" if it was actually used as an index.","commit_id":"1175f6b061f7294cabae7e72dd18df0758e45c21"},{"author":{"_account_id":14676,"name":"Kairat Kushaev","email":"kkushaev@mirantis.com","username":"kairat_kushaev"},"change_message_id":"22388cbb4da2867325b6b2087482c5fc98730965","unresolved":false,"context_lines":[{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        self.check_auth_params()"},{"line_number":127,"context_line":"        auth_url \u003d self.creds[\u0027auth_url\u0027]"},{"line_number":128,"context_line":"        for i in range(self.MAX_REDIRECTS):"},{"line_number":129,"context_line":"            try:"},{"line_number":130,"context_line":"                _authenticate(auth_url)"},{"line_number":131,"context_line":"            except exception.AuthorizationRedirect as e:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_74c70046","line":128,"in_reply_to":"9a8ffd7b_7727a464","updated":"2015-12-03 15:31:19.000000000","message":"Thanks for suggestion. Will update soon.","commit_id":"1175f6b061f7294cabae7e72dd18df0758e45c21"}]}