)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"8cb9c5b6b020bc135c15fd745ab9416b93d5eba9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update requests-mock syntax"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With requests-mock 0.5 we can do away with register_uri(method, ..) and"},{"line_number":10,"context_line":"just use the method name as a function. This feels much cleaner and"},{"line_number":11,"context_line":"looks more like requests syntax."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a961159_27ef41b2","line":9,"updated":"2014-12-08 00:29:01.000000000","message":"(note) requirements has requests-mock\u003e\u003d0.5.1 already.","commit_id":"63faf567f88ad2f3861bda77f4d6238ec8acd50e"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"f3d3a312ef14bd14081cf38bdaa81a29098b590b","unresolved":false,"context_lines":[{"line_number":10,"context_line":"just use the method name as a function. This feels much cleaner and"},{"line_number":11,"context_line":"looks more like requests syntax."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"In the same update there is query string decoding included in the history, so we no longer have to manually parse the query strings out of the url. "},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I43d31576d15b4be72350bebf00733c08a7fb3e6c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a961159_37560807","line":13,"updated":"2014-12-11 10:09:44.000000000","message":"Too many symbols per line","commit_id":"ba09c6c8253e9814a57bffa53d11516b7ae79de5"}],"keystoneclient/tests/test_auth_token_middleware.py":[{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"9a9f072158f01cf128ccd9c0f4fed1b1fd491226","unresolved":false,"context_lines":[{"line_number":294,"context_line":"                          status_code\u003d300)"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        self.requests.post(\"%s/v2.0/tokens\" % BASE_URI,"},{"line_number":297,"context_line":"                           text\u003dFAKE_ADMIN_TOKEN)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        self.token_id \u003d self.examples.VALID_DIABLO_TOKEN"},{"line_number":300,"context_line":"        token_response \u003d self.examples.JSON_TOKEN_RESPONSES[self.token_id]"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_b5a2ed96","line":297,"updated":"2014-12-11 12:55:20.000000000","message":"These 2 lines may be a single line","commit_id":"ba09c6c8253e9814a57bffa53d11516b7ae79de5"}],"keystoneclient/tests/v3/utils.py":[{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"a5665a1088947c5faabec8bcfa9dc7a6dd3d6c38","unresolved":false,"context_lines":[{"line_number":251,"context_line":"        self.assertEqual(len(ref_list), len(returned_list))"},{"line_number":252,"context_line":"        [self.assertIsInstance(r, self.model) for r in returned_list]"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        # register_uri doesn\u0027t match the querystring component, so we have to"},{"line_number":255,"context_line":"        # explicitly test the querystring component passed by the manager"},{"line_number":256,"context_line":"        parts \u003d urlparse.urlparse(self.requests.last_request.url)"},{"line_number":257,"context_line":"        qs_args \u003d urlparse.parse_qs(parts.query)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_27ac9335","line":254,"updated":"2014-11-19 08:53:06.000000000","message":"Please, fix this comment. I think that it won\u0027t be clear to someone new that requests.get is is a wrapper for requests.register_uri.","commit_id":"2b5e329e75bd9849c63401aabb3bae28a136fa9b"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"8cb9c5b6b020bc135c15fd745ab9416b93d5eba9","unresolved":false,"context_lines":[{"line_number":251,"context_line":"        self.assertEqual(len(ref_list), len(returned_list))"},{"line_number":252,"context_line":"        [self.assertIsInstance(r, self.model) for r in returned_list]"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        qs_args_expected \u003d expected_query or filter_kwargs"},{"line_number":255,"context_line":"        for key, value in six.iteritems(qs_args_expected):"},{"line_number":256,"context_line":"            self.assertIn(key, self.requests.last_request.qs)"},{"line_number":257,"context_line":"            # The querystring value is a list. Note we convert the value to a"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a961159_a73a5134","line":254,"updated":"2014-12-08 00:29:01.000000000","message":"this change is unrelated and should be in a separate commit, or mentioned in the commit message.","commit_id":"63faf567f88ad2f3861bda77f4d6238ec8acd50e"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"9a9f072158f01cf128ccd9c0f4fed1b1fd491226","unresolved":false,"context_lines":[{"line_number":253,"context_line":""},{"line_number":254,"context_line":"        qs_args_expected \u003d expected_query or filter_kwargs"},{"line_number":255,"context_line":"        for key, value in six.iteritems(qs_args_expected):"},{"line_number":256,"context_line":"            self.assertIn(key, self.requests.last_request.qs)"},{"line_number":257,"context_line":"            # The querystring value is a list. Note we convert the value to a"},{"line_number":258,"context_line":"            # string and lower, as the query string is always a string and the"},{"line_number":259,"context_line":"            # filter_kwargs may contain non-string values, for example a"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_b5daede5","line":256,"range":{"start_line":256,"start_character":31,"end_line":256,"end_character":60},"updated":"2014-12-11 12:55:20.000000000","message":"IMHO, self.requests.last_request.qs is a candidate to extract a variable","commit_id":"ba09c6c8253e9814a57bffa53d11516b7ae79de5"}]}