)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"3f041e5872e48fa4defaff7edf5960c96795c08c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fd28f02d_06550f45","updated":"2024-06-20 19:34:07.000000000","message":"I wanted to look into how to add a check for this issue, but since setuptools are already included in test-requirements, all the unit tests couldn\u0027t detect this. It also hadn\u0027t helped to remove it there, since it is also implicitly required by testtools. So just doing the change proposed here should be the best option afaict.","commit_id":"b8019d70ca51d4b4611578f6972023baa6a2092d"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"80ba0ad676b3b67ef0f64e46845a00dfb7d6cf13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e4b1cefd_7c18738f","updated":"2024-06-20 23:19:59.000000000","message":"Looks good to me","commit_id":"cfdb4fa162a96032565a61c71f9243900da37f1e"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"46b42d795a53db398bd4d84f0455b6c1e769855b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f1268689_54d41a4c","updated":"2024-06-21 03:19:03.000000000","message":"this makes sense to me too","commit_id":"cfdb4fa162a96032565a61c71f9243900da37f1e"}],"test-requirements.txt":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"80ba0ad676b3b67ef0f64e46845a00dfb7d6cf13","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"# NOTE: These are requirements for testing the requirements project only"},{"line_number":5,"context_line":"#       See global-requirements for the actual requirements list"},{"line_number":6,"context_line":"stestr\u003e\u003d1.0.0  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"b2fb766f_00043f5c","side":"PARENT","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":52},"updated":"2024-06-20 23:19:59.000000000","message":"nit: Why\u0027d you remove this comment?","commit_id":"79b6d5f681aa120c4972c63392fb9293de5fcab4"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"7a113543aada72d772883a8166abbdfc7d7c164b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"# NOTE: These are requirements for testing the requirements project only"},{"line_number":5,"context_line":"#       See global-requirements for the actual requirements list"},{"line_number":6,"context_line":"stestr\u003e\u003d1.0.0  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"3a801e25_bad8041a","side":"PARENT","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":52},"in_reply_to":"97b80fab_d6dbffd6","updated":"2024-06-24 01:44:47.000000000","message":"Acknowledged","commit_id":"79b6d5f681aa120c4972c63392fb9293de5fcab4"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"002d4ea4488619e06a529a3456f10be6b9b0aad1","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"# NOTE: These are requirements for testing the requirements project only"},{"line_number":5,"context_line":"#       See global-requirements for the actual requirements list"},{"line_number":6,"context_line":"stestr\u003e\u003d1.0.0  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":2,"id":"97b80fab_d6dbffd6","side":"PARENT","line":3,"range":{"start_line":1,"start_character":0,"end_line":3,"end_character":52},"in_reply_to":"b2fb766f_00043f5c","updated":"2024-06-21 06:10:48.000000000","message":"Because afaict this is no longer true for mildly modern pip versions and it has been removed from most other repos. I can revert if you don\u0027t want to change it, though.\n\nSee e.g. I743674eaf684be9d8b6255e90edc82de0cd80734","commit_id":"79b6d5f681aa120c4972c63392fb9293de5fcab4"}]}