)]}'
{".pre-commit-config.yaml":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"7eaef115b7fa8852d56cbf1a2e940f8a0480691a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"repos:"},{"line_number":3,"context_line":"  - repo: https://github.com/pre-commit/pre-commit-hooks"},{"line_number":4,"context_line":"    rev: v4.6.0"},{"line_number":5,"context_line":"    hooks:"},{"line_number":6,"context_line":"      - id: trailing-whitespace"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"caea8a28_4f736641","line":3,"updated":"2024-07-03 09:29:53.000000000","message":"+1, pre-commit is always good to have.","commit_id":"b0c4b4782de85b2fa7a10d076935e2e0f5f88a7a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"7eaef115b7fa8852d56cbf1a2e940f8a0480691a","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    hooks:"},{"line_number":26,"context_line":"      - id: hacking"},{"line_number":27,"context_line":"        additional_dependencies: []"},{"line_number":28,"context_line":"        exclude: \u0027^(doc|releasenotes|tools)/.*$\u0027"},{"line_number":29,"context_line":"  # TODO(stephenfin): Uncomment this"},{"line_number":30,"context_line":"  # - repo: https://github.com/sphinx-contrib/sphinx-lint"},{"line_number":31,"context_line":"  #   rev: v0.9.1"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4d29d259_a9cc1d27","line":28,"updated":"2024-07-03 09:29:53.000000000","message":"can you please add autopep8 too.\n\n```\n- repo: https://github.com/hhatto/autopep8\n    rev: v2.1.0\n    hooks:\n      - id: autopep8\n        files: \u0027^.*\\.py$\u0027\n```","commit_id":"b0c4b4782de85b2fa7a10d076935e2e0f5f88a7a"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"f09382ce70d31d0c8ed7fe00de4ea31564a67fcf","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    hooks:"},{"line_number":26,"context_line":"      - id: hacking"},{"line_number":27,"context_line":"        additional_dependencies: []"},{"line_number":28,"context_line":"        exclude: \u0027^(doc|releasenotes|tools)/.*$\u0027"},{"line_number":29,"context_line":"  # TODO(stephenfin): Uncomment this"},{"line_number":30,"context_line":"  # - repo: https://github.com/sphinx-contrib/sphinx-lint"},{"line_number":31,"context_line":"  #   rev: v0.9.1"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cec4e7e2_115aed54","line":28,"in_reply_to":"4d29d259_a9cc1d27","updated":"2024-07-08 09:51:42.000000000","message":"thanks","commit_id":"b0c4b4782de85b2fa7a10d076935e2e0f5f88a7a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"7eaef115b7fa8852d56cbf1a2e940f8a0480691a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b576eacb_ef9dcfbe","updated":"2024-07-03 09:29:53.000000000","message":"a request to add autopep8 and ask to update the commit msg for change (because seems unrelated to pre-commit).\nelse lgtm.","commit_id":"b0c4b4782de85b2fa7a10d076935e2e0f5f88a7a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"efd59a31c8541c1342f5833d363266a66af4c9c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"935a9048_5ffc7ccc","updated":"2024-10-01 20:10:43.000000000","message":"lgtm","commit_id":"094867041d67098b69ab66ae604ea4059ae19e56"}],"placement/tests/functional/gabbits/allocation-candidates-member-of.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6a8dab2ad8133428a18d1cb9db1acad485695603","unresolved":true,"context_lines":[{"line_number":107,"context_line":"  status: 200"},{"line_number":108,"context_line":"  response_json_paths:"},{"line_number":109,"context_line":"      $.allocation_requests.`len`: 2"},{"line_number":110,"context_line":"  status: 200"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"- name: verify microversion fail for multiple member_of params"},{"line_number":113,"context_line":"  GET: /allocation_candidates?resources\u003dVCPU:1,MEMORY_MB:1024,DISK_GB:100\u0026member_of\u003d$ENVIRON[\u0027AGGA_UUID\u0027]\u0026member_of\u003d$ENVIRON[\u0027AGGB_UUID\u0027]"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7e716f54_7018aff7","side":"PARENT","line":110,"updated":"2024-07-20 07:35:17.000000000","message":"this is removed because its a duplicate of line 107","commit_id":"1a7a807592567bed34fe3576311b3289c30c6df9"}],"placement/tests/functional/gabbits/microversion.yaml":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"7eaef115b7fa8852d56cbf1a2e940f8a0480691a","unresolved":true,"context_lines":[{"line_number":35,"context_line":"  response_json_paths:"},{"line_number":36,"context_line":"      $.errors.[0].title: Not Acceptable"},{"line_number":37,"context_line":"      $.errors.[0].max_version: /^\\d+\\.\\d+$/"},{"line_number":38,"context_line":"      $.errors.[0].min_version: /^\\d+\\.\\d+$/"},{"line_number":39,"context_line":"  response_strings:"},{"line_number":40,"context_line":"      - \"Unacceptable version header: 0.5\""},{"line_number":41,"context_line":"  response_json_paths:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f9b309a2_4e8b631e","side":"PARENT","line":38,"updated":"2024-07-03 09:29:53.000000000","message":"I did not get why this remove and remove in placement/tests/functional/gabbits/allocation-candidates-member-of.yaml is added.\n\ncan you please tell the reason in commit msg ?","commit_id":"51cbfeedba812132ca34839fd43405dbc2b88111"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6a8dab2ad8133428a18d1cb9db1acad485695603","unresolved":true,"context_lines":[{"line_number":35,"context_line":"  response_json_paths:"},{"line_number":36,"context_line":"      $.errors.[0].title: Not Acceptable"},{"line_number":37,"context_line":"      $.errors.[0].max_version: /^\\d+\\.\\d+$/"},{"line_number":38,"context_line":"      $.errors.[0].min_version: /^\\d+\\.\\d+$/"},{"line_number":39,"context_line":"  response_strings:"},{"line_number":40,"context_line":"      - \"Unacceptable version header: 0.5\""},{"line_number":41,"context_line":"  response_json_paths:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"346abf8a_c3346db2","side":"PARENT","line":38,"in_reply_to":"f9b309a2_4e8b631e","updated":"2024-07-20 07:35:17.000000000","message":"it conflicts with line 41 and given the test pass it was the correct one to remove","commit_id":"51cbfeedba812132ca34839fd43405dbc2b88111"}]}