)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"593452b8deddad4a0240b2de0d44c93641a9f0b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cb0cf364_a756ceb6","updated":"2023-02-17 15:27:40.000000000","message":"Fast approving as this is pretty trivial","commit_id":"b4be3a46032bd20c83847dd4ec32c49feddc5768"}],"tox.ini":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"15ed60ecad980cd2c76519f8373096eb492d3540","unresolved":true,"context_lines":[{"line_number":2,"context_line":"minversion \u003d 3.2.0"},{"line_number":3,"context_line":"envlist \u003d py3,pep8"},{"line_number":4,"context_line":"basepython \u003d python3"},{"line_number":5,"context_line":"ignore_basepython_conflict \u003d true"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"},{"line_number":8,"context_line":"deps \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"f614dc2f_678a09ca","line":5,"updated":"2023-02-17 11:19:12.000000000","message":"Can we remove these? \u0027basepython\u0027 is in the wrong place so this was never doing anything anyway","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"593452b8deddad4a0240b2de0d44c93641a9f0b3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"minversion \u003d 3.2.0"},{"line_number":3,"context_line":"envlist \u003d py3,pep8"},{"line_number":4,"context_line":"basepython \u003d python3"},{"line_number":5,"context_line":"ignore_basepython_conflict \u003d true"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[testenv]"},{"line_number":8,"context_line":"deps \u003d"}],"source_content_type":"text/x-properties","patch_set":1,"id":"b0f3faf6_7503e410","line":5,"in_reply_to":"f614dc2f_678a09ca","updated":"2023-02-17 15:27:40.000000000","message":"Done","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"15ed60ecad980cd2c76519f8373096eb492d3540","unresolved":true,"context_lines":[{"line_number":21,"context_line":"commands \u003d {posargs}"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[testenv:cover]"},{"line_number":24,"context_line":"commands \u003d python setup.py test --coverage --coverage-package-name\u003doslo_serialization --testr-args\u003d\u0027{posargs}\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"[testenv:docs]"},{"line_number":27,"context_line":"usedevelop \u003d false"}],"source_content_type":"text/x-properties","patch_set":1,"id":"39ed566f_75f8e6bf","line":24,"updated":"2023-02-17 11:19:12.000000000","message":"Somewhat unrelated, but this will not be supported in the near future. setuptools is removing support for distutils-style commands like this. We should probably use something like:\n\n  [testenv:cover]\n  setenv \u003d\n    PYTHON\u003dcoverage run --source oslo_concurrency --parallel-mode\n  commands \u003d\n    stestr run {posargs}\n    coverage combine\n    coverage html -d cover\n    coverage xml -o cover/coverage.xml\n\ninstead.","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"593452b8deddad4a0240b2de0d44c93641a9f0b3","unresolved":false,"context_lines":[{"line_number":21,"context_line":"commands \u003d {posargs}"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"[testenv:cover]"},{"line_number":24,"context_line":"commands \u003d python setup.py test --coverage --coverage-package-name\u003doslo_serialization --testr-args\u003d\u0027{posargs}\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"[testenv:docs]"},{"line_number":27,"context_line":"usedevelop \u003d false"}],"source_content_type":"text/x-properties","patch_set":1,"id":"327bed68_e06e0b77","line":24,"in_reply_to":"39ed566f_75f8e6bf","updated":"2023-02-17 15:27:40.000000000","message":"Done","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"15ed60ecad980cd2c76519f8373096eb492d3540","unresolved":true,"context_lines":[{"line_number":24,"context_line":"commands \u003d python setup.py test --coverage --coverage-package-name\u003doslo_serialization --testr-args\u003d\u0027{posargs}\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"[testenv:docs]"},{"line_number":27,"context_line":"usedevelop \u003d false"},{"line_number":28,"context_line":"allowlist_externals \u003d rm"},{"line_number":29,"context_line":"deps \u003d"},{"line_number":30,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"}],"source_content_type":"text/x-properties","patch_set":1,"id":"f30f47f4_5104d5f1","line":27,"updated":"2023-02-17 11:19:12.000000000","message":"Not necessary. Nothing is setting this to true","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"593452b8deddad4a0240b2de0d44c93641a9f0b3","unresolved":false,"context_lines":[{"line_number":24,"context_line":"commands \u003d python setup.py test --coverage --coverage-package-name\u003doslo_serialization --testr-args\u003d\u0027{posargs}\u0027"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"[testenv:docs]"},{"line_number":27,"context_line":"usedevelop \u003d false"},{"line_number":28,"context_line":"allowlist_externals \u003d rm"},{"line_number":29,"context_line":"deps \u003d"},{"line_number":30,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"}],"source_content_type":"text/x-properties","patch_set":1,"id":"11dda56f_d395b28a","line":27,"in_reply_to":"f30f47f4_5104d5f1","updated":"2023-02-17 15:27:40.000000000","message":"Done","commit_id":"bb706d49b9394b45e7ebd07db79d71a764ef22db"}]}