)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6841847b171f96c1466c4d62a2f553dc58e7daca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2c9d9b89_b601fc82","updated":"2025-11-12 18:35:24.000000000","message":"recheck failure on base patch","commit_id":"1239238da2640a9656e67667a4d0a6f6ca146703"}],"pyproject.toml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"369604b91dfee91ac306d721fca0b0a276a419f5","unresolved":true,"context_lines":[{"line_number":50,"context_line":"line-length \u003d 79"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"[tool.ruff.lint]"},{"line_number":53,"context_line":"select \u003d [\"C4\", \"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"S\", \"U\"]"},{"line_number":54,"context_line":"ignore \u003d ["},{"line_number":55,"context_line":"    # we only use asserts for type narrowing"},{"line_number":56,"context_line":"    \"S101\","}],"source_content_type":"text/x-toml","patch_set":1,"id":"4ba9cf2e_d15819c6","line":53,"range":{"start_line":53,"start_character":51,"end_line":53,"end_character":52},"updated":"2025-11-11 16:29:15.000000000","message":"I wonder if this needs to be UP ?","commit_id":"1239238da2640a9656e67667a4d0a6f6ca146703"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"76ee7484c53fd6f488f1ed863d5c66c3a3d6e575","unresolved":false,"context_lines":[{"line_number":50,"context_line":"line-length \u003d 79"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"[tool.ruff.lint]"},{"line_number":53,"context_line":"select \u003d [\"C4\", \"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"S\", \"U\"]"},{"line_number":54,"context_line":"ignore \u003d ["},{"line_number":55,"context_line":"    # we only use asserts for type narrowing"},{"line_number":56,"context_line":"    \"S101\","}],"source_content_type":"text/x-toml","patch_set":1,"id":"7f31e49e_4d36e3d7","line":53,"range":{"start_line":53,"start_character":51,"end_line":53,"end_character":52},"in_reply_to":"4ba9cf2e_d15819c6","updated":"2025-11-11 16:35:41.000000000","message":"Discussed on IRC but ruff accepts partial matches. That\u0027s how we e.g. enable `E4`, `E5`, `E7` and `E9` but not (a theoretical?) `E3`. We could also enable all `E` codes with just `E`. There would only be issues if they introduced a new `U[^P]` code class.\n\nThis is what happens if you introduce an invalid error.\n\n```\n❯ ruff check .\nruff failed\n  Cause: Failed to parse /home/stephenfin/Development/openstack/stevedore/pyproject.toml\n  Cause: TOML parse error at line 53, column 51\n   |\n53 | select \u003d [\"C4\", \"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"S\", \"V\"]\n   |                                                   ^^^\nUnknown rule selector: `V`\n```","commit_id":"1239238da2640a9656e67667a4d0a6f6ca146703"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"49c23a1243d349bbdbc1c43d5aa9eb45ca65225f","unresolved":false,"context_lines":[{"line_number":50,"context_line":"line-length \u003d 79"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"[tool.ruff.lint]"},{"line_number":53,"context_line":"select \u003d [\"C4\", \"E4\", \"E5\", \"E7\", \"E9\", \"F\", \"S\", \"U\"]"},{"line_number":54,"context_line":"ignore \u003d ["},{"line_number":55,"context_line":"    # we only use asserts for type narrowing"},{"line_number":56,"context_line":"    \"S101\","}],"source_content_type":"text/x-toml","patch_set":1,"id":"8edcf6c0_98dcae63","line":53,"range":{"start_line":53,"start_character":51,"end_line":53,"end_character":52},"in_reply_to":"4ba9cf2e_d15819c6","updated":"2025-11-11 16:34:52.000000000","message":"We agreed this is not urgent because of partial match. I\u0027d prefer using more specific pattern, though (so may update this later in a follow-up).","commit_id":"1239238da2640a9656e67667a4d0a6f6ca146703"}]}
