)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7b3aae2b99da03dea231a1868775bad86087a5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bb5653b7_271e803a","updated":"2026-02-09 12:50:19.000000000","message":"Looks good with just two (identical) nits. Fix those and I\u0027m +2","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c207d12be1c30c1a4cc96bec26edea5872e68d18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d4c02ca0_7aa4d492","updated":"2026-02-09 16:59:08.000000000","message":"Thanks for the review :-)","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"676ae671effeb9363ac2ef5bcd08977a301509da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1c180e88_7b2ea21d","updated":"2026-02-09 17:11:32.000000000","message":"This will fail pep8, but we\u0027re [fixing that currently](https://review.opendev.org/c/openstack/openstacksdk/+/976152). Feel free to recheck once that\u0027s merged.","commit_id":"9054ed5da170945071f6ec960f18537d16b55a4e"}],"osc_lib/tests/test_shell.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7b3aae2b99da03dea231a1868775bad86087a5c","unresolved":true,"context_lines":[{"line_number":328,"context_line":"            )"},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"        # Since we mock cliff\u0027s app _shell.command_manager is not set"},{"line_number":331,"context_line":"        # self.assertIsInstance(_shell.command_manager, commandmanager.CommandManager)"},{"line_number":332,"context_line":"        self.assertTrue(_shell.dump_stack_trace)"},{"line_number":333,"context_line":"        self.assertDictEqual({}, _shell.api_version)"},{"line_number":334,"context_line":"        self.assertListEqual([], _shell.command_options)"}],"source_content_type":"text/x-python","patch_set":2,"id":"224187db_9a57e23c","line":331,"updated":"2026-02-09 12:50:19.000000000","message":"Could you replace this with an `assertIsNone` or at least drop the commented out code?","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c207d12be1c30c1a4cc96bec26edea5872e68d18","unresolved":false,"context_lines":[{"line_number":328,"context_line":"            )"},{"line_number":329,"context_line":""},{"line_number":330,"context_line":"        # Since we mock cliff\u0027s app _shell.command_manager is not set"},{"line_number":331,"context_line":"        # self.assertIsInstance(_shell.command_manager, commandmanager.CommandManager)"},{"line_number":332,"context_line":"        self.assertTrue(_shell.dump_stack_trace)"},{"line_number":333,"context_line":"        self.assertDictEqual({}, _shell.api_version)"},{"line_number":334,"context_line":"        self.assertListEqual([], _shell.command_options)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f446c66_51bbdbbb","line":331,"in_reply_to":"224187db_9a57e23c","updated":"2026-02-09 16:59:08.000000000","message":"Done","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"c7b3aae2b99da03dea231a1868775bad86087a5c","unresolved":true,"context_lines":[{"line_number":379,"context_line":"            )"},{"line_number":380,"context_line":""},{"line_number":381,"context_line":"        # Since we mock cliff\u0027s app _shell.command_manager is not set"},{"line_number":382,"context_line":"        # self.assertIsInstance(_shell.command_manager, commandmanager.CommandManager)"},{"line_number":383,"context_line":"        self.assertTrue(_shell.dump_stack_trace)"},{"line_number":384,"context_line":"        self.assertDictEqual({}, _shell.api_version)"},{"line_number":385,"context_line":"        self.assertListEqual([], _shell.command_options)"}],"source_content_type":"text/x-python","patch_set":2,"id":"6514d42e_e0a5ccf3","line":382,"updated":"2026-02-09 12:50:19.000000000","message":"as [above](https://review.opendev.org/c/openstack/osc-lib/+/975784/2/osc_lib/tests/test_shell.py#331).","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c207d12be1c30c1a4cc96bec26edea5872e68d18","unresolved":false,"context_lines":[{"line_number":379,"context_line":"            )"},{"line_number":380,"context_line":""},{"line_number":381,"context_line":"        # Since we mock cliff\u0027s app _shell.command_manager is not set"},{"line_number":382,"context_line":"        # self.assertIsInstance(_shell.command_manager, commandmanager.CommandManager)"},{"line_number":383,"context_line":"        self.assertTrue(_shell.dump_stack_trace)"},{"line_number":384,"context_line":"        self.assertDictEqual({}, _shell.api_version)"},{"line_number":385,"context_line":"        self.assertListEqual([], _shell.command_options)"}],"source_content_type":"text/x-python","patch_set":2,"id":"94c218d9_7a2f2ba6","line":382,"in_reply_to":"6514d42e_e0a5ccf3","updated":"2026-02-09 16:59:08.000000000","message":"Done","commit_id":"3f17b580641ae36e0fe9d3547d0c5cd0c51f8896"}]}