)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"d68d45043928f7ad1b4c6be7b66c69be1744b23f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d0f66cab_1ace1fc8","updated":"2023-02-16 09:02:43.000000000","message":"LGTM!","commit_id":"919210c386a970ea0b28e4e7df3bb783c9f3895c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"2bbe613466688149b35dc7137f44b7b0032cdacb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5be739d5_e00a5607","updated":"2023-03-23 14:34:10.000000000","message":"Looks great!","commit_id":"919210c386a970ea0b28e4e7df3bb783c9f3895c"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"dbeab34133d2c730fafe05ce873c99326aad411e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"671d03b4_5ae1c1ab","updated":"2023-02-09 14:31:09.000000000","message":"This LGTM, thanks for doing this. The refactor here all makes sense, and the structure looks a lot cleaner to me. There also will probably be a mild performance improvement in the loader from this change (not that I expect it to be noticeable in practice).","commit_id":"919210c386a970ea0b28e4e7df3bb783c9f3895c"}],"reno/config.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"dbeab34133d2c730fafe05ce873c99326aad411e","unresolved":true,"context_lines":[{"line_number":219,"context_line":"]"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"class Config(object):"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    _OPTS \u003d {o.name: o for o in _OPTIONS}"},{"line_number":225,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"4e7f3ad9_efa603b6","side":"PARENT","line":222,"updated":"2023-02-09 14:31:09.000000000","message":"Unrelated to this patch, this makes me think we probably should run the whole repo through `pyupgrade` at some point to upgrade all the syntax to the minimum supported python version. I expect there\u0027s probably a lot of this kind of syntax leftover from when we still supported Python 2.7 (and older versions of python3).","commit_id":"04233e0eae9d688a3eb0630443a30b860c04601e"},{"author":{"_account_id":35758,"name":"Eric Arellano","display_name":"Eric Arellano","email":"ericarellano@me.com","username":"arellano"},"change_message_id":"4bbc6130b1d3af249aab6ef168f0a092b2c5e1bf","unresolved":false,"context_lines":[{"line_number":219,"context_line":"]"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"class Config(object):"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    _OPTS \u003d {o.name: o for o in _OPTIONS}"},{"line_number":225,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f93e737_089ef7bc","side":"PARENT","line":222,"in_reply_to":"4e7f3ad9_efa603b6","updated":"2023-02-10 01:20:19.000000000","message":"Definitely agreed with using `pyupgrade`. I noticed several places that could be modernized.\n\nWe also still claim support for Python 3.6, which is past EOL. Upgrading to Python 3.7 would give nice benefits like dataclasses.","commit_id":"04233e0eae9d688a3eb0630443a30b860c04601e"}]}