)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"b5746b19d3f487bdb2c9c0eaac8ac89dc8e093ad","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Following OpenStack Style Guidelines:"},{"line_number":10,"context_line":"http://docs.openstack.org/developer/hacking/#dictionaries-lists"},{"line_number":11,"context_line":"It is not clear as using explicit dictionaries"},{"line_number":12,"context_line":"and help avoid some errors during refactoring."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Iac53f4ae1fcfa8c0953bf635880f9b883ca9592b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1ac06dbe_9418e91f","line":12,"updated":"2016-08-23 11:04:55.000000000","message":"nit: Commit message should mention where this change is taking place, as it looks like this is actually happening in the Cinder code, while it is really only in lintstack file.","commit_id":"63e7085ea12e9e810d89a7a266e6c94df8d3b64a"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"eb2ea166bc529846610e723d93df31457f9f139f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Following OpenStack Style Guidelines:"},{"line_number":10,"context_line":"http://docs.openstack.org/developer/hacking/#dictionaries-lists"},{"line_number":11,"context_line":"It is not clear as using explicit dictionaries"},{"line_number":12,"context_line":"and help avoid some errors during refactoring."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Iac53f4ae1fcfa8c0953bf635880f9b883ca9592b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1ac06dbe_667f3a15","line":12,"updated":"2016-08-24 09:28:08.000000000","message":"nit: Commit message should mention where this change is taking place, as it looks like this is actually happening in the Cinder code, while it is really only in lintstack file.","commit_id":"5e0194fc25d6315c4296414e05cee9fe46d4a945"}],"tools/lintstack.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"b5746b19d3f487bdb2c9c0eaac8ac89dc8e093ad","unresolved":false,"context_lines":[{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    def review_str(self):"},{"line_number":182,"context_line":"        return (\"File %s\\nLine %d:%s\\n\""},{"line_number":183,"context_line":"                \"%s: %s\" % (self.filename, self.lineno, self.line_content, self.code, self.message))"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"class ErrorKeys(object):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_140d19e8","line":183,"updated":"2016-08-23 11:04:55.000000000","message":"-1: It\u0027s a great idea changing this, but please don\u0027t use positional arguments for the formatting, it is less clear.\n\n return (\"File %(filename)s\\nLine %(lineno)d:%(line_content)s\\n\"\n         \"%(code)s: %(message)s\" %\n         {\u0027filename\u0027: self.filename,\n          \u0027lineno\u0027: self.lineno,\n          \u0027line_content\u0027: self.line_content,\n          \u0027code\u0027: self.code,\n          \u0027message\u0027: self.message)","commit_id":"63e7085ea12e9e810d89a7a266e6c94df8d3b64a"}]}