)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38496,"name":"Andressa Cabistani","display_name":"Andressa","email":"acabistani@gmail.com","username":"andressadotpy","status":"I\u0027m a Software Engineer at Red Hat and I love Open Source and connect with people! Feel free to DM through IRC, I\u0027ll be delighted to chat"},"change_message_id":"25b4276e82e0ed6233f5a9eaffc76a98cb2c710b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"eb020527_dec5ae10","updated":"2026-06-02 10:38:24.000000000","message":"This patch looks really good and ready\n\n- I believe the patch description and commit message are great\n- The new function is consistent with the one in Eventlet source code\n- With Claude help, I created 15 tests to make sure the function had good coverage and all tests passed\n- All existent unit tests are passing\n\nThe tests covered:\n\nUnit tests:\n  1. Regular socket shutdown - verifies shutdown(SHUT_RDWR) is called\n  2. SSL socket shutdown - verifies shutdown(SHUT_RDWR) is called\n  3. PyOpenSSL socket shutdown - verifies TypeError path (shutdown with no args)\n  4. ENOTCONN error suppressed - socket not connected\n  5. EBADF error suppressed - bad file descriptor\n  6. ENOTSOCK error suppressed - not a socket\n  7. Other OSErrors re-raised - validates only specific errors suppressed\n  8. EPERM error re-raised - confirms filtering logic\n  9. PyOpenSSL + ENOTCONN suppressed - combined edge case\n  10. PyOpenSSL + other error raised - error propagation on non-suppressed errors\n  11. Non-OSError exceptions raised - ValueError propagates correctly\n  12. All three suppressed errors tested in sequence - comprehensive validation\n\nIntegration Tests with Real Sockets (3 tests):\n  13. Real connected socket pair - actual socket.socketpair() shutdown\n  14. Already closed socket - EBADF suppressed in practice\n  15. Unconnected socket - ENOTCONN suppressed in practice","commit_id":"c7d10c175091a2a7d82755dd54474702c7de6d00"},{"author":{"_account_id":39146,"name":"Nathaniel Martes","display_name":"Nate Martes","email":"nmartes@NVIDIA.com","username":"nmartes"},"change_message_id":"35de2a5a05f398d6e69043bada8a4cd8147b9149","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"9e7ffd9d_82e1f15b","updated":"2026-07-08 17:11:09.000000000","message":"This patch looks good to me! I really like the patch description as-well.","commit_id":"e52d86190b2d4fcfa0fb6e77068928fc5954de74"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"1712f11cd835cd77b8ea69dd25887ac2e50f7c15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cd84bbb2_df1011e1","updated":"2026-08-28 13:03:59.000000000","message":"Self-approving on feature branch after discussing within the core reviewers team.","commit_id":"1755002882a8a3b1dc4713b8a8ebd4e72b484dc8"}]}
