)]}'
{"ironicclient/common/base.py":[{"author":{"_account_id":19213,"name":"Szymon Borkowski","email":"szymon.borkowski@intel.com","username":"sborkows"},"change_message_id":"39c1b6be763512cc1cb660bc54ad48efe1a3e6d8","unresolved":false,"context_lines":[{"line_number":180,"context_line":"    def _delete(self, resource_id, etag\u003dNone):"},{"line_number":181,"context_line":"        \"\"\"Delete a resource."},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"        :param resource_id: Resource identifier."},{"line_number":184,"context_line":"        \"\"\""},{"line_number":185,"context_line":"        self.api.raw_request(\u0027DELETE\u0027, self._path(resource_id))"},{"line_number":186,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_d1406838","line":183,"updated":"2016-12-19 13:58:17.000000000","message":"nit: Would be good to add here \u0027etag\u0027 param info too.","commit_id":"d3b84b4a3188cd71e5a8c9106030fbe7738362b8"},{"author":{"_account_id":19213,"name":"Szymon Borkowski","email":"szymon.borkowski@intel.com","username":"sborkows"},"change_message_id":"cecdd7a0bd1800cc02fcd3b5496da6e0d3d7462a","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        if body:"},{"line_number":192,"context_line":"            return self.resource_class(self, body)"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def _delete(self, resource_id, etag\u003dNone):"},{"line_number":195,"context_line":"        \"\"\"Delete a resource."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        :param resource_id: Resource identifier."}],"source_content_type":"text/x-python","patch_set":9,"id":"da4df55a_bc1b881a","line":194,"range":{"start_line":194,"start_character":35,"end_line":194,"end_character":44},"updated":"2016-12-23 10:26:18.000000000","message":"I have not noticed this before. I think this etag should be passed further to API in some way (L200).","commit_id":"4b4d34f1b2d951d879257f7f4799e3d046c56c1d"},{"author":{"_account_id":22724,"name":"Galyna Zholtkevych","email":"galynazholtkevych1991@gmail.com","username":"gzholtkevych"},"change_message_id":"00ac3da860910f6852204a24b4b8772db96b6c44","unresolved":false,"context_lines":[{"line_number":191,"context_line":"        if body:"},{"line_number":192,"context_line":"            return self.resource_class(self, body)"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def _delete(self, resource_id, etag\u003dNone):"},{"line_number":195,"context_line":"        \"\"\"Delete a resource."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"        :param resource_id: Resource identifier."}],"source_content_type":"text/x-python","patch_set":9,"id":"da4df55a_f747ea74","line":194,"range":{"start_line":194,"start_character":35,"end_line":194,"end_character":44},"in_reply_to":"da4df55a_bc1b881a","updated":"2016-12-23 11:37:25.000000000","message":"This is not completed yet. Update is under testing.\nthe patch has a title [WIP], but I will update workflow to make it more clearly","commit_id":"4b4d34f1b2d951d879257f7f4799e3d046c56c1d"}],"ironicclient/tests/unit/utils.py":[{"author":{"_account_id":19213,"name":"Szymon Borkowski","email":"szymon.borkowski@intel.com","username":"sborkows"},"change_message_id":"cecdd7a0bd1800cc02fcd3b5496da6e0d3d7462a","unresolved":false,"context_lines":[{"line_number":122,"context_line":"                         raw\u003draw,"},{"line_number":123,"context_line":"                         reason\u003d\u0027\u0027,"},{"line_number":124,"context_line":"                         encoding\u003d\u0027UTF-8\u0027)"},{"line_number":125,"context_line":"    response.text \u003d content"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    return response"},{"line_number":128,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"da4df55a_dc9bec70","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":27},"updated":"2016-12-23 10:26:18.000000000","message":"nit: We can put it into Mock constructor too.","commit_id":"4b4d34f1b2d951d879257f7f4799e3d046c56c1d"}],"ironicclient/v1/node.py":[{"author":{"_account_id":19213,"name":"Szymon Borkowski","email":"szymon.borkowski@intel.com","username":"sborkows"},"change_message_id":"39c1b6be763512cc1cb660bc54ad48efe1a3e6d8","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                                         limit\u003dlimit)"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"    def get(self, node_id, fields\u003dNone):"},{"line_number":195,"context_line":"        resource \u003d self._get(resource_id\u003dnode_id, fields\u003dfields)"},{"line_number":196,"context_line":"        return resource"},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"    def get_by_instance_uuid(self, instance_uuid, fields\u003dNone):"},{"line_number":199,"context_line":"        path \u003d \u0027?instance_uuid\u003d%s\u0027 % instance_uuid"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a6eadb0_712f94f6","line":196,"range":{"start_line":195,"start_character":8,"end_line":196,"end_character":23},"updated":"2016-12-19 13:58:17.000000000","message":"nit: I\u0027m wondering whether this has any other purpose than just introducing new variable. Or maybe it\u0027s just about the style.","commit_id":"d3b84b4a3188cd71e5a8c9106030fbe7738362b8"}]}