)]}'
{".pre-commit-config.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"aee03f15e7ab25949e18a199794aa81c4342accf","unresolved":true,"context_lines":[{"line_number":18,"context_line":"      - id: debug-statements"},{"line_number":19,"context_line":"      - id: check-yaml"},{"line_number":20,"context_line":"        files: .*\\.(yaml|yml)$"},{"line_number":21,"context_line":"  - repo: https://opendev.org/openstack/hacking"},{"line_number":22,"context_line":"    rev: 7.0.0"},{"line_number":23,"context_line":"    hooks:"},{"line_number":24,"context_line":"      - id: hacking"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"a85bae73_70faa6eb","line":21,"updated":"2025-07-02 17:30:09.000000000","message":"Its better if you install this from pypi then the CI caches can do their job. I think the old rule did install from pypi as there is no git repo info for hacking in it.","commit_id":"9956df0d19dc3dab8a0f855e7251f2fc1fb24b29"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"6118ac4fd9c00df5c27484bea1179b8be58a2ad3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2ba06fd5_3cc04325","updated":"2025-07-02 15:05:21.000000000","message":"I don\u0027t use the pre-commit tool so don\u0027t really care, but I assume the removed comment about pinned commit Ids being safer than tagged versions was wholly misguided?","commit_id":"9956df0d19dc3dab8a0f855e7251f2fc1fb24b29"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"d5ffbfa96deaeb74a06223d64de1bd061f117554","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"14b2007c_c4888e22","updated":"2025-07-02 18:18:53.000000000","message":"I\u0027m going to go ahead and +2 this because I think PBR is a small drop in the bucket here with pre commit behavior. That said if there is opportunity to think about how to run pre-commit in a way that is more CI cache friendly I think we should try and apply that more broadly.","commit_id":"9956df0d19dc3dab8a0f855e7251f2fc1fb24b29"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"aee03f15e7ab25949e18a199794aa81c4342accf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ce7948d_63932a7f","in_reply_to":"2ba06fd5_3cc04325","updated":"2025-07-02 17:30:09.000000000","message":"Note you do use pre-commit if you run tox -epep8 in this repo.","commit_id":"9956df0d19dc3dab8a0f855e7251f2fc1fb24b29"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"b037a1b9106847c97843b83469a446751c81bc1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"604f9143_7ae1dda2","in_reply_to":"7ce7948d_63932a7f","updated":"2025-07-03 09:35:53.000000000","message":"Yes. If I recall correctly, that was from @dtantsur@protonmail.com way back when and we\u0027ve stopped doing it. I get where he was coming from (assuming it was Dmitry) but tbh I see little difference in building and running a package from a `pypa` git repo rather than a pypa pip package. If the Git repo gets compromised, it\u0027s game over regardless given how most of these Python packages are built on GitHub itself now.","commit_id":"9956df0d19dc3dab8a0f855e7251f2fc1fb24b29"}]}