)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"dc1189eb6cedbaa7c3f4cfb250a5e1474fd83f74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4213ac64_d98cf0b2","updated":"2024-11-29 11:46:45.000000000","message":"This is required to suppprot python 3.12 which is part fo the required testing runtime for 2025.1 (epoxy)\n\nthis is in line with the other work done in other project to remove the distutils depency","commit_id":"be7ee7347ad8fd8f61441de0d49189b928fe5f0b"}],"watcherclient/common/httpclient.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"01fc10c18b2a3a220f32e89da2c1a069ad298329","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        if negotiated_ver \u003e max_ver:"},{"line_number":130,"context_line":"            negotiated_ver \u003d max_ver"},{"line_number":131,"context_line":"        if negotiated_ver \u003c min_ver:"},{"line_number":132,"context_line":"            negotiated_ver \u003d min_ver"},{"line_number":133,"context_line":"        # server handles microversions, but doesn\u0027t support"},{"line_number":134,"context_line":"        # the requested version, so try a negotiated version"},{"line_number":135,"context_line":"        self.api_version_select_state \u003d \u0027negotiated\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa75ec19_2d90cb2c","line":132,"updated":"2024-11-13 14:03:31.000000000","message":"i guess this is fine i would be inlcned ot do\n\nnegotiated_ver \u003d max(min_version, min(negotiated_ver,max_ver)) \ninstead of 2 ifs but its logcially the same at the end of the day and in python i dont think its meaning fully faster to use min of max like it would be in \nc/go/rust","commit_id":"be7ee7347ad8fd8f61441de0d49189b928fe5f0b"}]}