)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f0aaf072b8a4356988aaf37e2cdceebed452b4b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"35b3daa3_1481280e","updated":"2025-10-07 15:14:56.000000000","message":"this is perhaps more contentious.\n\nif we dont want to always use fstring and just use format and fstring where it makes sense im fine with not proceeding with this.","commit_id":"0439681e4a9a5eb9e3515cb1335ffadadff53d5c"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"47ed3210e7013e307d3e00fba7eb49f7b0e868b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bdb33ad3_713f862d","updated":"2025-10-21 18:49:44.000000000","message":"LGTM, thanks!","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3f62399451f98b5697ccff35edfb5ce9f5a51acc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bf3d0b56_7e0eef34","updated":"2025-10-10 14:28:11.000000000","message":"There is still a couple of .format() around, can\u0027t we get them all replaced?","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"9a8439a9bcf292b8d2c78471f47ebf8981db8773","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"50b74f73_33d62b14","updated":"2025-10-10 12:02:22.000000000","message":"lgtm, personally I prefer f-strings over `format`","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"}],"watcher/api/middleware/parsable_error.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3f62399451f98b5697ccff35edfb5ce9f5a51acc","unresolved":true,"context_lines":[{"line_number":79,"context_line":"                except et.ElementTree.ParseError as err:"},{"line_number":80,"context_line":"                    LOG.error(\u0027Error parsing HTTP response: %s\u0027, err)"},{"line_number":81,"context_line":"                    body \u003d [\u0027\u003cerror_message\u003e{}\u0027"},{"line_number":82,"context_line":"                            \u0027\u003c/error_message\u003e\u0027.format(state[\u0027status_code\u0027])]"},{"line_number":83,"context_line":"                state[\u0027headers\u0027].append((\u0027Content-Type\u0027, \u0027application/xml\u0027))"},{"line_number":84,"context_line":"            else:"},{"line_number":85,"context_line":"                app_iter \u003d [i.decode(\u0027utf-8\u0027) for i in app_iter]"}],"source_content_type":"text/x-python","patch_set":4,"id":"a16d8800_cd8d9976","line":82,"range":{"start_line":82,"start_character":28,"end_line":82,"end_character":75},"updated":"2025-10-10 14:28:11.000000000","message":"why not using f-strings here?","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"}],"watcher/tests/api/v1/test_actions.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"3f62399451f98b5697ccff35edfb5ce9f5a51acc","unresolved":true,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    def test_show(self):"},{"line_number":118,"context_line":"        action \u003d obj_utils.create_test_action(self.context, parents\u003dNone)"},{"line_number":119,"context_line":"        response \u003d self.get_json(\u0027/actions/{}\u0027.format(action[\u0027uuid\u0027]))"},{"line_number":120,"context_line":"        self.assertEqual(action.uuid, response[\u0027uuid\u0027])"},{"line_number":121,"context_line":"        self.assertEqual(action.action_type, response[\u0027action_type\u0027])"},{"line_number":122,"context_line":"        self.assertEqual(action.input_parameters, response[\u0027input_parameters\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"5bbcb375_c91daf84","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":70},"updated":"2025-10-10 14:28:11.000000000","message":"why not using f-strings here? and other below?","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"47ed3210e7013e307d3e00fba7eb49f7b0e868b3","unresolved":true,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    def test_show(self):"},{"line_number":118,"context_line":"        action \u003d obj_utils.create_test_action(self.context, parents\u003dNone)"},{"line_number":119,"context_line":"        response \u003d self.get_json(\u0027/actions/{}\u0027.format(action[\u0027uuid\u0027]))"},{"line_number":120,"context_line":"        self.assertEqual(action.uuid, response[\u0027uuid\u0027])"},{"line_number":121,"context_line":"        self.assertEqual(action.action_type, response[\u0027action_type\u0027])"},{"line_number":122,"context_line":"        self.assertEqual(action.input_parameters, response[\u0027input_parameters\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"95cdcfd6_f1df8f84","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":70},"in_reply_to":"010bee3f_56685ab7","updated":"2025-10-21 18:49:44.000000000","message":"ah ok, lets continue with this patch as is then. Thanks!","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fc671ac9a69349a075e5d6e9d0642d86dce8de3a","unresolved":true,"context_lines":[{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    def test_show(self):"},{"line_number":118,"context_line":"        action \u003d obj_utils.create_test_action(self.context, parents\u003dNone)"},{"line_number":119,"context_line":"        response \u003d self.get_json(\u0027/actions/{}\u0027.format(action[\u0027uuid\u0027]))"},{"line_number":120,"context_line":"        self.assertEqual(action.uuid, response[\u0027uuid\u0027])"},{"line_number":121,"context_line":"        self.assertEqual(action.action_type, response[\u0027action_type\u0027])"},{"line_number":122,"context_line":"        self.assertEqual(action.input_parameters, response[\u0027input_parameters\u0027])"}],"source_content_type":"text/x-python","patch_set":4,"id":"010bee3f_56685ab7","line":119,"range":{"start_line":119,"start_character":0,"end_line":119,"end_character":70},"in_reply_to":"5bbcb375_c91daf84","updated":"2025-10-21 18:19:34.000000000","message":"im not sure but i suspect its related to he min python version\n\nthere are some places in py 3.10 where fstrings are not allowed or the tool just cant convert yet.","commit_id":"9b5c8c1c4f66ec3f9e2fa6415ec41cebfa12e96a"}]}