)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"9cf2353fd9ae620b7af40c846a632ace4f9a6d1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"866e53dc_6e2939c7","updated":"2023-09-22 14:01:57.000000000","message":"this will need a respin but yes we shoudl update this perodically\n\nand the hacking change is importat to do in lockstep with enableing a new hackign version in general to keep both the same","commit_id":"fe63e8f7acc18938a86a9e900ee5da1757ab6b19"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"5807f3febb16bbabcce619c026e25fe1edc4fe5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c95923e8_d63d9064","updated":"2023-10-02 14:09:10.000000000","message":"I have serious concerns with pre-commit hooks and I\u0027d prefer us to rather use pre-push hooks if you really want to not ask CI to verify for this.","commit_id":"964c902287f875e0454f9abbc6ec44c26494685d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"602978574ae394fe1cbd0bee4e41bfad5c906c87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c5928d0_58d7b123","updated":"2023-12-05 12:28:06.000000000","message":"Sean had +2\u0027d this previously and Sylvain has added his doc. This is breaking me locally since the version of flake8 we\u0027re currently using is incompatible with Python 3.12 so I\u0027m going to push this through","commit_id":"964c902287f875e0454f9abbc6ec44c26494685d"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"9356568a653d075f1a996ac85232fc20452f2701","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c05bec1d_7e4d921d","in_reply_to":"8af68eb7_803cd1e3","updated":"2023-10-02 15:30:20.000000000","message":"Ok, based on what we discussed on IRC, I can remove my objection, provided we clarify on [1] that pre-commit is absolutely optional in our development process and could be skipped.","commit_id":"964c902287f875e0454f9abbc6ec44c26494685d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1e665c0d5992c31f97281c4e189725cb1b34c51a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8af68eb7_803cd1e3","in_reply_to":"c95923e8_d63d9064","updated":"2023-10-02 14:56:02.000000000","message":"Huh? We are asking CI for this. That\u0027s what the `pep8` job is for. If you mean that we\u0027re not using the exact same configuration, I have addressed this in a later patch in this series ([here](https://review.opendev.org/c/openstack/nova/+/896227/2?usp\u003drelated-change)). Is there something else?","commit_id":"964c902287f875e0454f9abbc6ec44c26494685d"}]}