)]}'
{"tempest/services/identity/v3/json/services_client.py":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"87f17fdfeb5284188357806ac6923785e17a133d","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        body \u003d json.loads(body)"},{"line_number":58,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def delete_service(self, service_id):"},{"line_number":61,"context_line":"        url \u003d \"services/\" + service_id"},{"line_number":62,"context_line":"        resp, body \u003d self.delete(url)"},{"line_number":63,"context_line":"        self.expected_success(204, resp.status)"},{"line_number":64,"context_line":"        return rest_client.ResponseBody(resp, body)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_6c00e166","line":61,"range":{"start_line":60,"start_character":6,"end_line":61,"end_character":14},"updated":"2016-08-01 17:12:36.000000000","message":"nit: Add method doc string","commit_id":"b643462dd66297388a02bd66f53e99adea87b6f0"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"87f17fdfeb5284188357806ac6923785e17a133d","unresolved":false,"context_lines":[{"line_number":63,"context_line":"        self.expected_success(204, resp.status)"},{"line_number":64,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    def list_services(self **params):"},{"line_number":67,"context_line":"        \"\"\"List services."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        Available params: see http://developer.openstack.org/"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_ececd1bc","line":66,"range":{"start_line":66,"start_character":24,"end_line":66,"end_character":28},"updated":"2016-08-01 17:12:36.000000000","message":"missing a comma.\nFor consistency purposes: sed \u0027s/**params/**kwargs/g\u0027","commit_id":"b643462dd66297388a02bd66f53e99adea87b6f0"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8d7e9e9c996d8a838225dacc732c899ca7e74c03","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        self.expected_success(204, resp.status)"},{"line_number":65,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def list_services(self, **params):"},{"line_number":68,"context_line":"        \"\"\"List services."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        Available params: see http://developer.openstack.org/"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_2404a4be","line":67,"updated":"2016-08-04 08:25:35.000000000","message":"Is there any particular reason of this naming \u0027params\u0027? The others are \"kwargs\" in this class, though.\nI don\u0027t stick on this, just curious.\nBut anyway, we can update this with following patches if we need.","commit_id":"ef301d80632adf28823fea82aca0b84fbd67e4c1"}]}