)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"680b6393ffc9470d804bd931b545ba3de55087d8","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Note that \u0027assertEquals\u0027/\u0027assertNotEquals\u0027, which is checked for by one"},{"line_number":12,"context_line":"test, is still present in some codebases, despite it being removed in"},{"line_number":13,"context_line":"Python 3. This is because this variant is added by testttools, which"},{"line_number":14,"context_line":"adds its own deprecation warning that we don\u0027t need to duplicate."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I3f15376db78886b7eecad3fdfdd363c6ea5d19c6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"714f8d5d_3a0a0f18","line":13,"range":{"start_line":13,"start_character":51,"end_line":13,"end_character":61},"updated":"2024-08-02 08:53:34.000000000","message":"nit: testtools?","commit_id":"8c6e0b85a1ad673b113a54f9b1e6ed17e87ddbe9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f4cfd0f4fff49f451c6f0d8cb436bc5edb20eac2","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Note that \u0027assertEquals\u0027/\u0027assertNotEquals\u0027, which is checked for by one"},{"line_number":12,"context_line":"test, is still present in some codebases, despite it being removed in"},{"line_number":13,"context_line":"Python 3. This is because this variant is added by testttools, which"},{"line_number":14,"context_line":"adds its own deprecation warning that we don\u0027t need to duplicate."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I3f15376db78886b7eecad3fdfdd363c6ea5d19c6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"34357132_d10e2df6","line":13,"range":{"start_line":13,"start_character":51,"end_line":13,"end_character":61},"in_reply_to":"714f8d5d_3a0a0f18","updated":"2024-08-02 10:25:45.000000000","message":"Done","commit_id":"8c6e0b85a1ad673b113a54f9b1e6ed17e87ddbe9"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"90c692fd5a0de02860776f3672578a7c2b153b71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"52685140_31a736df","updated":"2024-07-20 00:50:06.000000000","message":"lgtm","commit_id":"8c6e0b85a1ad673b113a54f9b1e6ed17e87ddbe9"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"250b6b0fb8afc9d4b1dc635312f7bf6b61393208","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4a3df3df_76974e2c","updated":"2024-08-05 10:13:29.000000000","message":"thx, re-using previous +2 to approve","commit_id":"e8a45225e1620ea7f6f2bb4c0f45cea79071ea61"}],"hacking/checks/except_checks.py":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"680b6393ffc9470d804bd931b545ba3de55087d8","unresolved":true,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"def is_none(node):"},{"line_number":92,"context_line":"    \"\"\"Check whether an AST node corresponds to None.\"\"\""},{"line_number":93,"context_line":"    return isinstance(node, ast.NameConstant) and node.value is None"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"4121e761_370c8c54","line":92,"updated":"2024-08-02 08:53:34.000000000","message":"these changes are unrelated, maybe at least mention them in the commit message? or better do a dedicated patch?","commit_id":"8c6e0b85a1ad673b113a54f9b1e6ed17e87ddbe9"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f4cfd0f4fff49f451c6f0d8cb436bc5edb20eac2","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"def is_none(node):"},{"line_number":92,"context_line":"    \"\"\"Check whether an AST node corresponds to None.\"\"\""},{"line_number":93,"context_line":"    return isinstance(node, ast.NameConstant) and node.value is None"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"0c02181f_dd10ad96","line":92,"in_reply_to":"4121e761_370c8c54","updated":"2024-08-02 10:25:45.000000000","message":"Done","commit_id":"8c6e0b85a1ad673b113a54f9b1e6ed17e87ddbe9"}]}