)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d9251f81eb5bae537c730f42c660a342b7dbf3b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ae999338_44b096ac","updated":"2025-05-26 13:52:27.000000000","message":"LGTM, test is checking the current behavior:\n```\ntest_launch_action_plan_failed_actions   pass\n```","commit_id":"b36ba8399e4be1362ce41f8809c4f43df05a4dc6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6d34c994310847d829e9a8035c67598129ee4802","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d2515d97_a8af3543","updated":"2025-05-26 11:40:40.000000000","message":"if you need to rebase this pleas address the nits in line","commit_id":"b36ba8399e4be1362ce41f8809c4f43df05a4dc6"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"aef860112414063d03ec40eaa23d542630554965","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a6d7ad13_f649eeb9","updated":"2025-05-22 11:45:42.000000000","message":"thank you for adding more testcases","commit_id":"b36ba8399e4be1362ce41f8809c4f43df05a4dc6"}],"watcher/tests/applier/action_plan/test_default_action_handler.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6d34c994310847d829e9a8035c67598129ee4802","unresolved":true,"context_lines":[{"line_number":143,"context_line":"            mock.call(self.context, self.action_plan,"},{"line_number":144,"context_line":"                      action\u003dobjects.fields.NotificationAction.EXECUTION,"},{"line_number":145,"context_line":"                      phase\u003dobjects.fields.NotificationPhase.END)]"},{"line_number":146,"context_line":"        # (amoralej) the actual action_plan.state should beh FAILED. I am"},{"line_number":147,"context_line":"        # setting it to SUCCEEDDED and will change it in the fixing change."},{"line_number":148,"context_line":"        self.assertEqual(ap_objects.State.SUCCEEDED, self.action_plan.state)"},{"line_number":149,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"363340b0_3cc17c7a","line":146,"range":{"start_line":146,"start_character":10,"end_line":146,"end_character":20},"updated":"2025-05-26 11:40:40.000000000","message":"```suggestion\n        # FIXME(amoralej): the actual action_plan.state should beh FAILED. I am\n```\n\nour upstream convention is \n\n``# [FIXME|TODO|NOTE]\\(name\\)\\:``","commit_id":"b36ba8399e4be1362ce41f8809c4f43df05a4dc6"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6d34c994310847d829e9a8035c67598129ee4802","unresolved":true,"context_lines":[{"line_number":143,"context_line":"            mock.call(self.context, self.action_plan,"},{"line_number":144,"context_line":"                      action\u003dobjects.fields.NotificationAction.EXECUTION,"},{"line_number":145,"context_line":"                      phase\u003dobjects.fields.NotificationPhase.END)]"},{"line_number":146,"context_line":"        # (amoralej) the actual action_plan.state should beh FAILED. I am"},{"line_number":147,"context_line":"        # setting it to SUCCEEDDED and will change it in the fixing change."},{"line_number":148,"context_line":"        self.assertEqual(ap_objects.State.SUCCEEDED, self.action_plan.state)"},{"line_number":149,"context_line":"        self.assertEqual("}],"source_content_type":"text/x-python","patch_set":1,"id":"2e810535_3bd83dba","line":146,"range":{"start_line":146,"start_character":57,"end_line":146,"end_character":60},"updated":"2025-05-26 11:40:40.000000000","message":"nit: im not sure why codespell did not detect this but perhaps its becasue its a comment.\n\n\n\n```suggestion\n        # (amoralej) the actual action_plan.state should be FAILED. I am\n```","commit_id":"b36ba8399e4be1362ce41f8809c4f43df05a4dc6"}]}