)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3e27609cdac8bb0f8095969389a1949d6308ac78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4e080a58_b34a2f30","updated":"2025-07-31 05:46:32.000000000","message":"Thank you for your review, Iury. Uploading a revised version.","commit_id":"3f2491429b48f22435f0012e01ae960cd202c016"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2162f360345247d863fce78d75df155c0cc1e66f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3cb75e2f_46c49376","updated":"2025-08-01 06:31:14.000000000","message":"Thank you for your review Julia, new version coming shortly.","commit_id":"58b935d9771abe12369bade1cfe741cb0278363e"}],"sushy/connector.py":[{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"4bb7aeff93c258593444664e0fcaf0b5661b346d","unresolved":true,"context_lines":[{"line_number":402,"context_line":"            headers_if_match \u003d headers.get(\u0027If-Match\u0027)"},{"line_number":403,"context_line":"            if headers_if_match \u003d\u003d \u0027\u0027:"},{"line_number":404,"context_line":"                del headers[\u0027If-Match\u0027]"},{"line_number":405,"context_line":"        if etag is not None and etag:"},{"line_number":406,"context_line":"            if not headers:"},{"line_number":407,"context_line":"                headers \u003d {}"},{"line_number":408,"context_line":"            headers[\u0027If-Match\u0027] \u003d etag"}],"source_content_type":"text/x-python","patch_set":1,"id":"9ea1bebd_3dda2733","line":405,"range":{"start_line":405,"start_character":8,"end_line":405,"end_character":37},"updated":"2025-07-31 02:06:23.000000000","message":"you can probably change this to just `if etag` no needto check for None and also if has a value.","commit_id":"1369b50967091179778a7aeca4dab14c9dbeda5b"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"3e27609cdac8bb0f8095969389a1949d6308ac78","unresolved":false,"context_lines":[{"line_number":402,"context_line":"            headers_if_match \u003d headers.get(\u0027If-Match\u0027)"},{"line_number":403,"context_line":"            if headers_if_match \u003d\u003d \u0027\u0027:"},{"line_number":404,"context_line":"                del headers[\u0027If-Match\u0027]"},{"line_number":405,"context_line":"        if etag is not None and etag:"},{"line_number":406,"context_line":"            if not headers:"},{"line_number":407,"context_line":"                headers \u003d {}"},{"line_number":408,"context_line":"            headers[\u0027If-Match\u0027] \u003d etag"}],"source_content_type":"text/x-python","patch_set":1,"id":"b0fec351_b4d79fe3","line":405,"range":{"start_line":405,"start_character":8,"end_line":405,"end_character":37},"in_reply_to":"9ea1bebd_3dda2733","updated":"2025-07-31 05:46:32.000000000","message":"Good point. This was a leftover from the previous revision. I also made the headers part consistent with this recommendation.","commit_id":"1369b50967091179778a7aeca4dab14c9dbeda5b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"bd1b1d0b448e373d106157ac3f060794b95be1ae","unresolved":true,"context_lines":[{"line_number":400,"context_line":"        # disregard it."},{"line_number":401,"context_line":"        if headers:"},{"line_number":402,"context_line":"            headers_if_match \u003d headers.get(\u0027If-Match\u0027)"},{"line_number":403,"context_line":"            if headers_if_match \u003d\u003d \u0027\u0027:"},{"line_number":404,"context_line":"                del headers[\u0027If-Match\u0027]"},{"line_number":405,"context_line":"        if etag:"},{"line_number":406,"context_line":"            if not headers:"}],"source_content_type":"text/x-python","patch_set":3,"id":"e33d0424_5658b14a","line":403,"updated":"2025-07-31 16:10:11.000000000","message":"since you don\u0027t re-use headers_if_match, I think you should do the get and comparison all on one line.","commit_id":"58b935d9771abe12369bade1cfe741cb0278363e"},{"author":{"_account_id":32177,"name":"Jacob Anders","email":"jacob-anders-dev@proton.me","username":"janders"},"change_message_id":"2162f360345247d863fce78d75df155c0cc1e66f","unresolved":false,"context_lines":[{"line_number":400,"context_line":"        # disregard it."},{"line_number":401,"context_line":"        if headers:"},{"line_number":402,"context_line":"            headers_if_match \u003d headers.get(\u0027If-Match\u0027)"},{"line_number":403,"context_line":"            if headers_if_match \u003d\u003d \u0027\u0027:"},{"line_number":404,"context_line":"                del headers[\u0027If-Match\u0027]"},{"line_number":405,"context_line":"        if etag:"},{"line_number":406,"context_line":"            if not headers:"}],"source_content_type":"text/x-python","patch_set":3,"id":"eee7cbf8_ce3ffcb1","line":403,"in_reply_to":"e33d0424_5658b14a","updated":"2025-08-01 06:31:14.000000000","message":"Good point Julia, uploading revised version. Thank you.","commit_id":"58b935d9771abe12369bade1cfe741cb0278363e"}]}