)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0d12a44e3fc252772ea8895fc8b919be2f335482","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"328ff8b6_c5ef5ea8","updated":"2026-07-07 10:30:54.000000000","message":"there is nothing wrong her ebut i want to see if i can resude the formating impact by removing the extra commans","commit_id":"5900cc4672a6333ece8391ecf36d770b0a730a30"}],"cyborg_tempest_plugin/services/cyborg_rest_client.py":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"1b6cc3a6252754ff8983423380e4f1058979ae79","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"def get_auth_provider(credentials, scope\u003d\u0027project\u0027):"},{"line_number":154,"context_line":"    default_params \u003d {"},{"line_number":155,"context_line":"        \u0027disable_ssl_certificate_validation\u0027: CONF.identity.disable_ssl_certificate_validation,"},{"line_number":156,"context_line":"        \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":157,"context_line":"        \u0027trace_requests\u0027: CONF.debug.trace_requests,"},{"line_number":158,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"45ddc81a_23cf12c8","line":155,"updated":"2026-07-06 23:09:36.000000000","message":"The skip-magic-trailing-comma\u003dtrue ruff format setting caused a previously multi-line dict entry to be collapsed onto a single 95-character line, violating the project\u0027s configured line-length of 79 and exceeding the prior state where no Python line exceeded 79 chars.\n\n**Severity**: SUGGESTION | **Confidence**: 0.9\n\n**Benefit**: The only Python line in the repository exceeding the configured 79-char limit was introduced by automated formatting. No active linter will flag it, so it will persist and set a precedent for future edits. The commit message also does not mention the skip-magic-trailing-comma setting, so its impa...\n\n**Recommendation**:\nEither manually wrap the line back to two lines (as it was before reformatting), or reconsider the skip-magic-trailing-comma \u003d true setting. If the setting is intentional, consider adding E501 to the ruff lint select list to prevent future violations.","commit_id":"5900cc4672a6333ece8391ecf36d770b0a730a30"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0d12a44e3fc252772ea8895fc8b919be2f335482","unresolved":true,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"def get_auth_provider(credentials, scope\u003d\u0027project\u0027):"},{"line_number":154,"context_line":"    default_params \u003d {"},{"line_number":155,"context_line":"        \u0027disable_ssl_certificate_validation\u0027: CONF.identity.disable_ssl_certificate_validation,"},{"line_number":156,"context_line":"        \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":157,"context_line":"        \u0027trace_requests\u0027: CONF.debug.trace_requests,"},{"line_number":158,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":3,"id":"399fc87a_2a811b8a","line":155,"in_reply_to":"45ddc81a_23cf12c8","updated":"2026-07-07 10:30:54.000000000","message":"im not actully sure that this is what happend","commit_id":"5900cc4672a6333ece8391ecf36d770b0a730a30"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0d12a44e3fc252772ea8895fc8b919be2f335482","unresolved":true,"context_lines":[{"line_number":154,"context_line":"    default_params \u003d {"},{"line_number":155,"context_line":"        \u0027disable_ssl_certificate_validation\u0027: CONF.identity.disable_ssl_certificate_validation,"},{"line_number":156,"context_line":"        \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":157,"context_line":"        \u0027trace_requests\u0027: CONF.debug.trace_requests,"},{"line_number":158,"context_line":"    }"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    if isinstance(credentials, auth.KeystoneV3Credentials):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf9ed7fa_2b6cc7ff","line":157,"range":{"start_line":157,"start_character":51,"end_line":157,"end_character":52},"updated":"2026-07-07 10:30:54.000000000","message":"however this comma shoudl not be there","commit_id":"5900cc4672a6333ece8391ecf36d770b0a730a30"},{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"6c0bfd9cd1da90eb4264f1b05672dc3ff7982f9e","unresolved":false,"context_lines":[{"line_number":171,"context_line":""},{"line_number":172,"context_line":"def get_auth_provider(credentials, scope\u003d\u0027project\u0027):"},{"line_number":173,"context_line":"    default_params \u003d {"},{"line_number":174,"context_line":"        \u0027disable_ssl_certificate_validation\u0027: CONF.identity.disable_ssl_certificate_validation,"},{"line_number":175,"context_line":"        \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":176,"context_line":"        \u0027trace_requests\u0027: CONF.debug.trace_requests,"},{"line_number":177,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":4,"id":"e9f0d143_a61a84f3","line":174,"updated":"2026-09-18 16:50:17.000000000","message":"Removing `flake8 {posargs}` from tox -e pep8 drops the only E501 (79-char) gate in the project: the new hacking hook runs flake8 with `select \u003d H` (no E codes), the ruff lint select list (E4, E7, E9, F, G, LOG, S, UP, W, C90) contains no E5 rules, and ruff format cannot split unbreakable attribute chains even with line-length \u003d 79. As a result the reformat in this change produced a 95-character line in get_auth_provider() with nothing to catch it, and it is now the only over-limit line in the repository, which was previously flake8 E501-clean.\n\n**Severity**: WARNING | **Confidence**: 0.92\n\n**Impact**: The project\u0027s enforced 79-char style rule regresses to unenforced. This change itself lands a violating line, and future over-long lines will merge with no CI signal, steadily eroding the standard the repository previously held across every file. Runtime behavior is unaffected.\n\n**Suggestion**:\nRestore the gate and fix the single violation: add \"E501\" to [tool.ruff.lint] select in pyproject.toml (this line is the only offender repo-wide, so enabling it is otherwise clean), and shorten line 174, e.g. hoist `disable_ssl \u003d CONF.identity.disable_ssl_certificate_validation` above the dict and reference the local, since ruff format cannot split the attribute chain itself.","commit_id":"c14228bda7aa34e2604fef3c4b87369b5b339c39"}],"cyborg_tempest_plugin/tests/api/test_accelerator_request.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0d12a44e3fc252772ea8895fc8b919be2f335482","unresolved":true,"context_lines":[{"line_number":23,"context_line":"    {"},{"line_number":24,"context_line":"        \"resources:FPGA\": \"1\","},{"line_number":25,"context_line":"        \"trait:CUSTOM_FPGA_1\": \"required\","},{"line_number":26,"context_line":"        \"trait:CUSTOM_FUNCTION_ID_3AFB\": \"required\","},{"line_number":27,"context_line":"    }"},{"line_number":28,"context_line":"]"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"cc73acdb_ce31e36a","line":26,"range":{"start_line":26,"start_character":51,"end_line":26,"end_character":52},"updated":"2026-07-07 10:30:54.000000000","message":"i think this refly was caused by the addtion of the trailign comma\nwhic we shoud lnot do by default.","commit_id":"5900cc4672a6333ece8391ecf36d770b0a730a30"}]}