)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17717,"name":"liuke","email":"liuke2@huawei.com","username":"liuke"},"change_message_id":"3833fca1c677255c3b8ca5732901b92a5d56cf1c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_b840aea6","line":9,"range":{"start_line":9,"start_character":31,"end_line":9,"end_character":36},"updated":"2016-07-20 01:45:54.000000000","message":"\u0027Limit\u0027 should be a lowercase letter.","commit_id":"0d203da232ae869855524b9e8930122e983f704d"},{"author":{"_account_id":17717,"name":"liuke","email":"liuke2@huawei.com","username":"liuke"},"change_message_id":"3833fca1c677255c3b8ca5732901b92a5d56cf1c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_b8d26efe","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":45},"updated":"2016-07-20 01:45:54.000000000","message":"Missing \".\"","commit_id":"0d203da232ae869855524b9e8930122e983f704d"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"a1a191c9a6b02da89bc4911f3a052a296cb86779","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_4fc37d89","line":9,"range":{"start_line":9,"start_character":31,"end_line":9,"end_character":36},"in_reply_to":"dada55a8_b840aea6","updated":"2016-07-20 16:34:00.000000000","message":"It\u0027s actually the name of a class - which is always uppercase in Python","commit_id":"0d203da232ae869855524b9e8930122e983f704d"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"a1a191c9a6b02da89bc4911f3a052a296cb86779","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_afbcb906","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":45},"in_reply_to":"dada55a8_b8d26efe","updated":"2016-07-20 16:34:00.000000000","message":"Done","commit_id":"0d203da232ae869855524b9e8930122e983f704d"},{"author":{"_account_id":17717,"name":"liuke","email":"liuke2@huawei.com","username":"liuke"},"change_message_id":"3833fca1c677255c3b8ca5732901b92a5d56cf1c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_f8dce613","line":10,"range":{"start_line":10,"start_character":65,"end_line":10,"end_character":69},"updated":"2016-07-20 01:45:54.000000000","message":"Missing \".\"","commit_id":"0d203da232ae869855524b9e8930122e983f704d"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"a1a191c9a6b02da89bc4911f3a052a296cb86779","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Added unit test coverage to test_limits.py for api v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added one new test for invalid Limit creation"},{"line_number":10,"context_line":"Refactored a test from `assert isinstance` to `self.assertIsInstance`"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I06b9dd7146c6d1a438bca745bca45b66350acbf9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dada55a8_ef3871a2","line":10,"range":{"start_line":10,"start_character":65,"end_line":10,"end_character":69},"in_reply_to":"dada55a8_f8dce613","updated":"2016-07-20 16:34:00.000000000","message":"Done","commit_id":"0d203da232ae869855524b9e8930122e983f704d"}],"cinder/tests/unit/api/v2/test_limits.py":[{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"81691d0409788dcc89f94acf0374af18eeac9fed","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2012 OpenStack Foundation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_19a2cf4b","line":1,"updated":"2016-06-28 13:35:23.000000000","message":"Why modify 2011 to 2012?","commit_id":"df3866abaf83b619d5222a97baf6782ec3b95082"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"81691d0409788dcc89f94acf0374af18eeac9fed","unresolved":false,"context_lines":[{"line_number":346,"context_line":"                                            \u0027(POST, /bar*, /bar.*, 5, second);\u0027"},{"line_number":347,"context_line":"                                            \u0027(Say, /derp*, /derp.*, 1, day)\u0027)"},{"line_number":348,"context_line":"        except ValueError as e:"},{"line_number":349,"context_line":"            assert False, six.text_type(e)"},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        # Make sure the number of returned limits are correct"},{"line_number":352,"context_line":"        self.assertEqual(4, len(l))"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_797793d4","line":349,"updated":"2016-06-28 13:35:23.000000000","message":"I think you can modify this to assertFalse too.","commit_id":"df3866abaf83b619d5222a97baf6782ec3b95082"}]}