)]}'
{".pre-commit-config.yaml":[{"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":"b5056f3ce8873e016890ed6b93234ec8b8fde6d3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"        args: [\u0027--fix\u0027, \u0027lf\u0027]"},{"line_number":11,"context_line":"        exclude: \u0027.*\\.(svg)$\u0027"},{"line_number":12,"context_line":"      - id: fix-byte-order-marker"},{"line_number":13,"context_line":"      - id: check-executables-have-shebangs"},{"line_number":14,"context_line":"      - id: check-shebang-scripts-are-executable"},{"line_number":15,"context_line":"      - id: check-yaml"},{"line_number":16,"context_line":"        files: .*\\.(yaml|yml)$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"84eeb971_03e68e87","line":13,"updated":"2026-07-06 22:38:00.000000000","message":"The commit message enumerates 12 hooks but the actual .pre-commit-config.yaml defines 14. The hooks check-executables-have-shebangs and check-shebang-scripts-are-executable are configured but not mentioned in the commit body.\n\n**Severity**: SUGGESTION | **Confidence**: 0.9\n\n**Benefit**: Minor documentation inaccuracy. Reviewers and future maintainers reading the commit history will have an incomplete understanding of which hooks are active. Low operational impact since the actual config file is authoritative.\n\n**Recommendation**:\nUpdate the commit message to include check-executables-have-shebangs and check-shebang-scripts-are-executable in the hook list, or use a general statement such as \u0027and other standard pre-commit-hooks\u0027 instead of enumerating each hook.","commit_id":"e5f0855772bd497b866d760dad92a7e5f9f7d2ac"}],"/COMMIT_MSG":[{"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":"85772d9b7f8a18a5c17f25be1cf1a905d502c19a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     ae4fd72a (Refactor PCI lifecycle tests)"},{"line_number":2,"context_line":"Author:     Sean Mooney \u003cwork@seanmooney.info\u003e"},{"line_number":3,"context_line":"AuthorDate: 2026-06-08 20:13:56 +0100"},{"line_number":4,"context_line":"Commit:     Sean Mooney \u003cwork@seanmooney.info\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"46820654_35bca9a8","line":1,"updated":"2026-09-18 16:09:01.000000000","message":"The commit message enumerates the enabled hooks as: trailing-whitespace, mixed-line-ending, byte-order-marker, check-yaml, check-json, check-ast, check-added-large-files, check-case-conflict, detect-private-key, check-merge-conflict, debug-statements, check-docstring-first (12 hooks). The .pre-commit-config.yaml added by this commit actually enables 14 hooks; check-executables-have-shebangs and check-shebang-scripts-are-executable (.pre-commit-config.yaml:19-20) are missing from the list.\n\n**Severity**: SUGGESTION | **Confidence**: 0.85\n\n**Impact**: Readers auditing the change from its message alone (e.g., via git log when the file has since changed) would under-count the enabled checks and miss that shebang/executable-bit consistency is enforced by the pep8 gate, reducing the accuracy of the permanent project history.\n\n**Recommendation**:\nWhen next amending this commit (e.g., to extend the config with the planned ruff and hacking hooks), update the \u0027Hooks enabled:\u0027 list to also include check-executables-have-shebangs and check-shebang-scripts-are-executable, or reword it to reference the pre-commit-hooks suite generally so the list cannot drift.","commit_id":"12433b91922e55cc1f3ae780074803fdbef01f93"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b28751ac870bc9644aaeaa3f845605cc80ffd329","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"586111b9_1e300f03","updated":"2026-06-09 07:36:27.000000000","message":"+1 pre-commit hook is running with tox -e pep8 https://zuul.opendev.org/t/openstack/build/7f4df856e030475fafa77904dc9f799f/log/job-output.txt#970","commit_id":"7d0903e67806ca9520003aa771851a29d00b0aec"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"1b48bd8ff4f7ff367624795b60e6ceba526f26e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"591fe66b_7a90eefc","updated":"2026-07-22 15:42:50.000000000","message":"looks good to me, needs a rebase","commit_id":"e5f0855772bd497b866d760dad92a7e5f9f7d2ac"}]}