)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"430629f2316f4892226f484f53e0d880afbdb8db","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"e.g.: from assertTrue(sth \u003d\u003d None) to assertIsNone(*) or"},{"line_number":12,"context_line":"assertTrue(isinstance(inst, type)) to assertIsInstace(inst, type) or"},{"line_number":13,"context_line":"assertTrue(not sth) to assertFalse(sth)."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"The code gets more readable, and a better description will be shown on fail."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba5da102_ed7faa12","line":13,"updated":"2016-11-03 01:17:09.000000000","message":"NIT: you give examples here, but you never actually make any of the example changes. The only changes that are made are assertIn, assetNotIn, assertGreater and assertEqual. So you could be more specific :)  But hey it does explain why your making the change so good enough :)","commit_id":"300d388825c228ae1f51cdb8abcdf4c76ec24637"}],"test/probe/test_object_metadata_replication.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b20cf166f718b29194ae0ba0c9737b07862514a8","unresolved":false,"context_lines":[{"line_number":311,"context_line":"            self.assertIn(key, metadata)"},{"line_number":312,"context_line":"            self.assertEqual(metadata[key], usermeta[key])"},{"line_number":313,"context_line":"        for key in sysmeta2.keys():"},{"line_number":314,"context_line":"            self.assertIn(key, metadata, key)"},{"line_number":315,"context_line":"            self.assertEqual(metadata[key], sysmeta2[key])"},{"line_number":316,"context_line":"        self.brain.start_primary_half()"},{"line_number":317,"context_line":"        self.container_brain.start_primary_half()"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba5da102_87745736","line":314,"range":{"start_line":314,"start_character":39,"end_line":314,"end_character":44},"updated":"2016-11-02 18:18:36.000000000","message":"nit: We could probably drop the message arg now.","commit_id":"300d388825c228ae1f51cdb8abcdf4c76ec24637"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b20cf166f718b29194ae0ba0c9737b07862514a8","unresolved":false,"context_lines":[{"line_number":324,"context_line":"            self.assertIn(key, metadata)"},{"line_number":325,"context_line":"            self.assertEqual(metadata[key], usermeta[key])"},{"line_number":326,"context_line":"        for key in sysmeta2.keys():"},{"line_number":327,"context_line":"            self.assertIn(key, metadata, key)"},{"line_number":328,"context_line":"            self.assertEqual(metadata[key], sysmeta2[key])"},{"line_number":329,"context_line":"        self.brain.start_handoff_half()"},{"line_number":330,"context_line":"        self.container_brain.start_handoff_half()"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba5da102_0797673a","line":327,"range":{"start_line":327,"start_character":39,"end_line":327,"end_character":44},"updated":"2016-11-02 18:18:36.000000000","message":"Here, too.","commit_id":"300d388825c228ae1f51cdb8abcdf4c76ec24637"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b20cf166f718b29194ae0ba0c9737b07862514a8","unresolved":false,"context_lines":[{"line_number":379,"context_line":"        expected.update(usermeta)"},{"line_number":380,"context_line":"        expected.update(transient_sysmeta)"},{"line_number":381,"context_line":"        for key in expected.keys():"},{"line_number":382,"context_line":"            self.assertIn(key, metadata, key)"},{"line_number":383,"context_line":"            self.assertEqual(metadata[key], expected[key])"},{"line_number":384,"context_line":"        self.brain.start_primary_half()"},{"line_number":385,"context_line":"        self.container_brain.start_primary_half()"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba5da102_c7b72fd6","line":382,"range":{"start_line":382,"start_character":39,"end_line":382,"end_character":44},"updated":"2016-11-02 18:18:36.000000000","message":"And here.","commit_id":"300d388825c228ae1f51cdb8abcdf4c76ec24637"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b20cf166f718b29194ae0ba0c9737b07862514a8","unresolved":false,"context_lines":[{"line_number":389,"context_line":"        self.container_brain.stop_handoff_half()"},{"line_number":390,"context_line":"        metadata \u003d self._get_object_metadata()"},{"line_number":391,"context_line":"        for key in expected.keys():"},{"line_number":392,"context_line":"            self.assertIn(key, metadata, key)"},{"line_number":393,"context_line":"            self.assertEqual(metadata[key], expected[key])"},{"line_number":394,"context_line":"        self.brain.start_handoff_half()"},{"line_number":395,"context_line":"        self.container_brain.start_handoff_half()"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba5da102_07ae8780","line":392,"range":{"start_line":392,"start_character":39,"end_line":392,"end_character":44},"updated":"2016-11-02 18:18:36.000000000","message":"And here.","commit_id":"300d388825c228ae1f51cdb8abcdf4c76ec24637"}]}