)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"411e4de19a0a1810a5768f1542721c51e98cfa09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3898c75e_7e61a934","updated":"2022-07-27 16:39:23.000000000","message":"Thanks for the patch Clark, it makes sense to me!","commit_id":"55215ac11cc960c61d8efd3f742d31482b52272e"}],"playbooks/proposal/pre.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"411e4de19a0a1810a5768f1542721c51e98cfa09","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        name: ensure-tox"},{"line_number":82,"context_line":"      vars:"},{"line_number":83,"context_line":"        ensure_global_symlinks: true"},{"line_number":84,"context_line":"        ensure_pip_virtualenv_command: \u0027{{ _venv_command | default(omit) }}\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0e920d4c_41b20e2a","side":"PARENT","line":84,"updated":"2022-07-27 16:39:23.000000000","message":"I haven\u0027t used \u0027omit\u0027 so far, but google shows there are some confusions around its usage, so yes, omitting \u0027omit\u0027 (sorry o:)) might work :)","commit_id":"4616695be07dd5e4c3ec4db716792493cdc46cec"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"b43707574cc35e0dbb8cd356f69306b8d08e83c6","unresolved":true,"context_lines":[{"line_number":82,"context_line":"      vars:"},{"line_number":83,"context_line":"        ensure_global_symlinks: true"},{"line_number":84,"context_line":"        ensure_pip_virtualenv_command: \u0027{{ _venv_command }}\u0027"},{"line_number":85,"context_line":"      when: _venv_command is defined"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    - name: Install tox without venv command override"},{"line_number":88,"context_line":"      include_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3904f156_587fccf2","line":85,"updated":"2022-07-27 21:02:00.000000000","message":"I guess this only runs on bionic or focal, so we could have always set the venv command to \"/usr/bin/ptyhon3.8 ...\" for both; it would work equally.  probably more confusing","commit_id":"55215ac11cc960c61d8efd3f742d31482b52272e"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"411e4de19a0a1810a5768f1542721c51e98cfa09","unresolved":true,"context_lines":[{"line_number":89,"context_line":"        name: ensure-tox"},{"line_number":90,"context_line":"      vars:"},{"line_number":91,"context_line":"        ensure_global_symlinks: true"},{"line_number":92,"context_line":"      when: _venv_command is not defined"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2827999e_e0368760","line":92,"updated":"2022-07-27 16:39:23.000000000","message":"with this \u0027when\u0027 statements the intention seems clear","commit_id":"55215ac11cc960c61d8efd3f742d31482b52272e"}]}