)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"27d54e25ee19ae2408576c95b6eb2a5acfcef6e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"928ec754_256096c5","updated":"2025-10-01 10:10:59.000000000","message":"it\u0027s mainly removing old python idioms, like classes inheriting from `object` and passing the class name to `super`","commit_id":"e9da13935b9af4ddd42fa633f5248969538ca4ad"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"1c08c5be8614c777cdc5209e870d777621ad5fda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b7703f56_96f4c613","in_reply_to":"928ec754_256096c5","updated":"2025-10-01 10:50:58.000000000","message":"yep exactly with the flag i passed it is removing the code that existed only for python 2.7 compatibality","commit_id":"e9da13935b9af4ddd42fa633f5248969538ca4ad"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"5dc96a222d12b3640262b247db4e743e3526588e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"de388e70_1bb8903c","updated":"2025-10-06 20:00:35.000000000","message":"All changes lgtm, most of them are similar.\nFor reviewers: \u0027]\u0027 and PageDown are your friends here 😊","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"eff9fce95666c1a0cd6646822edb2e2aa98aecab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5c346967_68041c6d","updated":"2025-10-06 14:12:06.000000000","message":"recheck unreatled tempest failures\n\ntest_continuous_audit_zone_migration failed to generate a non empty action plan but i belive this is a known unstable test.\n\nwatcher-tempest-actuator failed because of https://bugs.launchpad.net/aodh/+bug/2126581","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"b2a33e20d210615c2878795a9ff21e1c97d48929","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"15ffa343_c80165fb","updated":"2025-10-07 07:46:23.000000000","message":"upgrading my vote, since there has been no negative feedback adding +W as well","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"}],"watcher/api/controllers/root.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8dde3e77f7f084ae9ef350e9b54d0e55d225570d","unresolved":true,"context_lines":[{"line_number":119,"context_line":"            args \u003d [self._default_version] + args"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        return super()._route(args, request)"},{"line_number":122,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"28a75e53_4d383ff3","line":122,"in_reply_to":"09e0abbd_42fd1cf6","updated":"2025-10-01 15:09:11.000000000","message":"sigh this was the merge conflict i had to rebase for...\nill rebase again and fix this once the last patch in the seriese reports","commit_id":"2dc6c225d0303b136257e63db748bea6624e4d9b"}],"watcher/common/utils.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"5dc96a222d12b3640262b247db4e743e3526588e","unresolved":true,"context_lines":[{"line_number":125,"context_line":"            if \"default\" in subschema and instance is not None:"},{"line_number":126,"context_line":"                instance.setdefault(prop, subschema[\"default\"])"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"            yield from validate_properties("},{"line_number":129,"context_line":"                validator, properties, instance, schema"},{"line_number":130,"context_line":"            )"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bb4707d0_1d71e970","line":128,"range":{"start_line":128,"start_character":11,"end_line":128,"end_character":24},"updated":"2025-10-06 20:00:35.000000000","message":"TIL","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8b123be58ccff4ae9b29487159a1987a9813d9e1","unresolved":true,"context_lines":[{"line_number":125,"context_line":"            if \"default\" in subschema and instance is not None:"},{"line_number":126,"context_line":"                instance.setdefault(prop, subschema[\"default\"])"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"            yield from validate_properties("},{"line_number":129,"context_line":"                validator, properties, instance, schema"},{"line_number":130,"context_line":"            )"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"69067c30_ed5968ff","line":128,"range":{"start_line":128,"start_character":11,"end_line":128,"end_character":24},"in_reply_to":"bb4707d0_1d71e970","updated":"2025-10-06 20:52:12.000000000","message":"ya we could not use this while we had py2.7 compat as this came in python 3.3\n\nhttps://peps.python.org/pep-0380/\n\ni actully tought it was in 3.5 but i guess that just because i didnt really use 3.3 or 3.4","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"}],"watcher/decision_engine/audit/continuous.py":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"5dc96a222d12b3640262b247db4e743e3526588e","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        return False"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    def do_execute(self, audit, request_context):"},{"line_number":84,"context_line":"        solution \u003d super()\\"},{"line_number":85,"context_line":"            .do_execute(audit, request_context)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        if audit.audit_type \u003d\u003d objects.audit.AuditType.CONTINUOUS.value:"}],"source_content_type":"text/x-python","patch_set":3,"id":"a3fa9ced_d0fde7d7","line":84,"range":{"start_line":84,"start_character":26,"end_line":84,"end_character":27},"updated":"2025-10-06 20:00:35.000000000","message":"we could remove this too","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8b123be58ccff4ae9b29487159a1987a9813d9e1","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        return False"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    def do_execute(self, audit, request_context):"},{"line_number":84,"context_line":"        solution \u003d super()\\"},{"line_number":85,"context_line":"            .do_execute(audit, request_context)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        if audit.audit_type \u003d\u003d objects.audit.AuditType.CONTINUOUS.value:"}],"source_content_type":"text/x-python","patch_set":3,"id":"f91b0b61_e30f1fdb","line":84,"range":{"start_line":84,"start_character":26,"end_line":84,"end_character":27},"in_reply_to":"a3fa9ced_d0fde7d7","updated":"2025-10-06 20:52:12.000000000","message":"yes we could. auto pep8 wont collapse this because both are valid but\nif we were using ruff it would have re flowed this.\n\nfor the most part i tied to keep this as effectively the output of the tool as clean up by pre-commit so that im not mixing manual changes with tool generated code.","commit_id":"a1fdc7a1edf9203153b8c2ba29f55d1ae04311b8"}]}