)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7da9cddccbdddfb133b64133ef424ce8bf3978cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"340a4e49_3fd15192","updated":"2025-09-12 16:51:11.000000000","message":"Gonna get this in since I want the type hints for things I\u0027m working on","commit_id":"5faff28d9bc13909008308377bc3ea67d68495ac"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"ca03ca9d6b522b18df97756d2e2be16124ca407e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"da8fbe8a_6c8861ef","updated":"2025-09-15 10:34:43.000000000","message":"recheck failure on base patch","commit_id":"5faff28d9bc13909008308377bc3ea67d68495ac"}],"oslo_utils/excutils.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"1e7f82dede2fe90e17475e17a827fb692e5c93f8","unresolved":true,"context_lines":[{"line_number":213,"context_line":"                \" to force the reraising of\""},{"line_number":214,"context_line":"            )"},{"line_number":215,"context_line":"        try:"},{"line_number":216,"context_line":"            if self.value is None:"},{"line_number":217,"context_line":"                if self.type_ is not None:"},{"line_number":218,"context_line":"                    self.value \u003d self.type_()"},{"line_number":219,"context_line":"            if ("},{"line_number":220,"context_line":"                self.value is not None"},{"line_number":221,"context_line":"                and self.value.__traceback__ is not self.tb"},{"line_number":222,"context_line":"            ):"},{"line_number":223,"context_line":"                raise self.value.with_traceback(self.tb)"},{"line_number":224,"context_line":"            if self.value is not None:"},{"line_number":225,"context_line":"                raise self.value"},{"line_number":226,"context_line":"        finally:"},{"line_number":227,"context_line":"            self.value \u003d None"},{"line_number":228,"context_line":"            self.tb \u003d None"}],"source_content_type":"text/x-python","patch_set":3,"id":"9b62dbb2_5cc75170","line":225,"range":{"start_line":216,"start_character":12,"end_line":225,"end_character":32},"updated":"2025-09-11 13:11:27.000000000","message":"I find this code is a bit hard to read, the previous version was more digest.\nWhat do you think of the following:\n\n```\nif self.value is None and self.type_ is not None:\n    self.value \u003d self.type_()\n\nif self.value is not None:\n    if self.value.__traceback__ is not self.tb:\n        raise self.value.with_traceback(self.tb)\n     \n    raise self.value\n```\n\nWe do not need to recheck if `value` is not `None`.\nThis snippet is a mix between the previous version and your improvement.\nIt makes readability a little clearer.","commit_id":"38d13fe9898636f833730686ba0153c332a07665"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e1d1628eced03f9f356803c31d9e9928bb1b811d","unresolved":false,"context_lines":[{"line_number":213,"context_line":"                \" to force the reraising of\""},{"line_number":214,"context_line":"            )"},{"line_number":215,"context_line":"        try:"},{"line_number":216,"context_line":"            if self.value is None:"},{"line_number":217,"context_line":"                if self.type_ is not None:"},{"line_number":218,"context_line":"                    self.value \u003d self.type_()"},{"line_number":219,"context_line":"            if ("},{"line_number":220,"context_line":"                self.value is not None"},{"line_number":221,"context_line":"                and self.value.__traceback__ is not self.tb"},{"line_number":222,"context_line":"            ):"},{"line_number":223,"context_line":"                raise self.value.with_traceback(self.tb)"},{"line_number":224,"context_line":"            if self.value is not None:"},{"line_number":225,"context_line":"                raise self.value"},{"line_number":226,"context_line":"        finally:"},{"line_number":227,"context_line":"            self.value \u003d None"},{"line_number":228,"context_line":"            self.tb \u003d None"}],"source_content_type":"text/x-python","patch_set":3,"id":"be9bd625_9359ddbc","line":225,"range":{"start_line":216,"start_character":12,"end_line":225,"end_character":32},"in_reply_to":"9b62dbb2_5cc75170","updated":"2025-09-11 16:55:10.000000000","message":"Works for me 👌","commit_id":"38d13fe9898636f833730686ba0153c332a07665"}]}