)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"e2dd0542936d719d5069da42fe89c4160e84347f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b86eaa4c_e66a5a01","updated":"2023-11-10 18:19:47.000000000","message":"Looks OK to me","commit_id":"167e3380c946ffb07622a8c8079fa1b22b8c5543"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"259ae78ff07309b5c064332d0b1bb5c2a7285b7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c25c7d07_d1e92f41","updated":"2023-10-02 16:03:35.000000000","message":"sure that ok enhancement to the doc to make in my view.","commit_id":"167e3380c946ffb07622a8c8079fa1b22b8c5543"}],"doc/source/contributor/development-environment.rst":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"574eecf6eb46978fe7b1485e26cbb50ac981b082","unresolved":false,"context_lines":[{"line_number":178,"context_line":"As a reminder, the hooks are optional and you are not enforced to run them."},{"line_number":179,"context_line":"You can either not install pre-commit or skip the hooks once by using the"},{"line_number":180,"context_line":"`--no-verify` flag on `git commit`."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"Using a remote debugger"},{"line_number":183,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":184,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"b839f8d8_d35eac62","line":181,"updated":"2023-11-01 10:11:39.000000000","message":"yeah, though it is very helpful and I think not everyone really know about it, because it required to install separately.\nI got to know, after Sean told me about it when I was getting too many errors in gate.","commit_id":"167e3380c946ffb07622a8c8079fa1b22b8c5543"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"fce602839e0167c68ec1086f04a3d0d67e6e17b0","unresolved":false,"context_lines":[{"line_number":178,"context_line":"As a reminder, the hooks are optional and you are not enforced to run them."},{"line_number":179,"context_line":"You can either not install pre-commit or skip the hooks once by using the"},{"line_number":180,"context_line":"`--no-verify` flag on `git commit`."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"Using a remote debugger"},{"line_number":183,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":184,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"b0fc30d0_1fad241f","line":181,"in_reply_to":"4ee98fb8_799f9ea8","updated":"2023-11-13 09:07:55.000000000","message":"ya thats partly because fast8 is deprecated since we dont want to have to maintian that for ever and pep8 actully check all files in the repo not just the ones that were chnaged in the current commits so it take a long time.\n\nyou can continue to use them but it will be faster to use pre-commit if you like that workflow","commit_id":"167e3380c946ffb07622a8c8079fa1b22b8c5543"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"e2dd0542936d719d5069da42fe89c4160e84347f","unresolved":false,"context_lines":[{"line_number":178,"context_line":"As a reminder, the hooks are optional and you are not enforced to run them."},{"line_number":179,"context_line":"You can either not install pre-commit or skip the hooks once by using the"},{"line_number":180,"context_line":"`--no-verify` flag on `git commit`."},{"line_number":181,"context_line":""},{"line_number":182,"context_line":"Using a remote debugger"},{"line_number":183,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":184,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"4ee98fb8_799f9ea8","line":181,"in_reply_to":"b839f8d8_d35eac62","updated":"2023-11-10 18:19:47.000000000","message":"FWIW there is a message output whenever \u0027tox -epep8\u0027 or \u0027tox -efast8\u0027 are run that mentions pre-commit along with instructions about how to use it [1], that\u0027s how I learned about it:\n\n```\n    echo \"Running flake8 on ${files}\"\n    echo \"\"\n    echo \"Consider using the \u0027pre-commit\u0027 tool instead.\"\n    echo \"\"\n    echo \"    pip install --user pre-commit\"\n    echo \"    pre-commit install --allow-missing-config\"\n    echo \"\"\n```\n\n[1] https://github.com/openstack/nova/blob/338ee65c4c0c4110f5f5a6a072164c90a9c0c880/tools/flake8wrap.sh#L15-L21","commit_id":"167e3380c946ffb07622a8c8079fa1b22b8c5543"}]}