)]}'
{"ironicclient/shell.py":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"65a3d376ffe7755c1d41cda0ab71f7e6e06f0ddf","unresolved":false,"context_lines":[{"line_number":22,"context_line":"import sys"},{"line_number":23,"context_line":"import re"},{"line_number":24,"context_line":"import pkgutil"},{"line_number":25,"context_line":"import os"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from keystoneauth1.loading import session as kasession"},{"line_number":28,"context_line":"from oslo_utils import encodeutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_be360150","line":25,"updated":"2017-01-20 15:46:25.000000000","message":"alphabetic order please","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"93ec66efc22a6c15197c41cc43b5b677d549f5df","unresolved":false,"context_lines":[{"line_number":22,"context_line":"import sys"},{"line_number":23,"context_line":"import re"},{"line_number":24,"context_line":"import pkgutil"},{"line_number":25,"context_line":"import os"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"from keystoneauth1.loading import session as kasession"},{"line_number":28,"context_line":"from oslo_utils import encodeutils"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_96cbdb52","line":25,"in_reply_to":"5a3905b3_be360150","updated":"2021-07-29 08:35:41.000000000","message":"Done","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"65a3d376ffe7755c1d41cda0ab71f7e6e06f0ddf","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from ironicclient.common.i18n import _"},{"line_number":36,"context_line":"from ironicclient.common import utils"},{"line_number":37,"context_line":"from ironicclient import exc"},{"line_number":38,"context_line":"from ironicclient.common.apiclient import exceptions"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"LATEST_API_VERSION \u003d (\u00271\u0027, \u0027latest\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_3e4bf1ca","line":38,"updated":"2017-01-20 15:46:25.000000000","message":"ditto, not in alphabetic order","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"93ec66efc22a6c15197c41cc43b5b677d549f5df","unresolved":false,"context_lines":[{"line_number":35,"context_line":"from ironicclient.common.i18n import _"},{"line_number":36,"context_line":"from ironicclient.common import utils"},{"line_number":37,"context_line":"from ironicclient import exc"},{"line_number":38,"context_line":"from ironicclient.common.apiclient import exceptions"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"LATEST_API_VERSION \u003d (\u00271\u0027, \u0027latest\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_16de0b11","line":38,"in_reply_to":"5a3905b3_3e4bf1ca","updated":"2021-07-29 08:35:41.000000000","message":"Done","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"65a3d376ffe7755c1d41cda0ab71f7e6e06f0ddf","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        return parser"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"    def get_available_major_versions(self):"},{"line_number":243,"context_line":"        matcher \u003d re.compile(r\"v[0-9]*$\")"},{"line_number":244,"context_line":"        submodules \u003d pkgutil.iter_modules([os.path.dirname(__file__)])"},{"line_number":245,"context_line":"        available_versions \u003d [name[1:] for loader, name, ispkg in submodules"},{"line_number":246,"context_line":"                              if matcher.search(name)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_beac41a1","line":243,"range":{"start_line":243,"start_character":37,"end_line":243,"end_character":38},"updated":"2017-01-20 15:46:25.000000000","message":"Seems like what we need here is \"^v[0-9]+$\" here, so that it does not match something that just ends with v :)","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"93ec66efc22a6c15197c41cc43b5b677d549f5df","unresolved":false,"context_lines":[{"line_number":240,"context_line":"        return parser"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"    def get_available_major_versions(self):"},{"line_number":243,"context_line":"        matcher \u003d re.compile(r\"v[0-9]*$\")"},{"line_number":244,"context_line":"        submodules \u003d pkgutil.iter_modules([os.path.dirname(__file__)])"},{"line_number":245,"context_line":"        available_versions \u003d [name[1:] for loader, name, ispkg in submodules"},{"line_number":246,"context_line":"                              if matcher.search(name)]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_76ebcfb0","line":243,"range":{"start_line":243,"start_character":37,"end_line":243,"end_character":38},"in_reply_to":"5a3905b3_beac41a1","updated":"2021-07-29 08:35:41.000000000","message":"Done","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"65a3d376ffe7755c1d41cda0ab71f7e6e06f0ddf","unresolved":false,"context_lines":[{"line_number":241,"context_line":""},{"line_number":242,"context_line":"    def get_available_major_versions(self):"},{"line_number":243,"context_line":"        matcher \u003d re.compile(r\"v[0-9]*$\")"},{"line_number":244,"context_line":"        submodules \u003d pkgutil.iter_modules([os.path.dirname(__file__)])"},{"line_number":245,"context_line":"        available_versions \u003d [name[1:] for loader, name, ispkg in submodules"},{"line_number":246,"context_line":"                              if matcher.search(name)]"},{"line_number":247,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_fef96981","line":244,"updated":"2017-01-20 15:46:25.000000000","message":"I don\u0027t quite like this, but do not see any better option :(","commit_id":"e14c42d3252b629515763a533fd48f14aaa32639"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"de0571d2555a28e9d8bd2a7160972600c72bc6d9","unresolved":false,"context_lines":[{"line_number":258,"context_line":"                                                            version, \u0027shell\u0027)"},{"line_number":259,"context_line":"        except ImportError as e:"},{"line_number":260,"context_line":"            msg \u003d _(\"Invalid client version \u0027%(version)s\u0027. \""},{"line_number":261,"context_line":"                    \"Major part must be one of: \u0027%(major)s\u0027\") % {"},{"line_number":262,"context_line":"                \"version\": version,"},{"line_number":263,"context_line":"                \"major\": \", \".join(self.get_available_major_versions())}"},{"line_number":264,"context_line":"            raise exceptions.UnsupportedVersion("},{"line_number":265,"context_line":"                _(\u0027%(message)s, error was: %(error)s\u0027) %"},{"line_number":266,"context_line":"                {\u0027message\u0027: msg, \u0027error\u0027: e})"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa31d9ce_f0111f53","line":263,"range":{"start_line":261,"start_character":64,"end_line":263,"end_character":72},"updated":"2017-02-15 14:13:54.000000000","message":"Weird formatting here.","commit_id":"97f82c5a46f63f99129bc3d504928e4c062095dd"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"3483fd76ac40d6b91b1716b676b1bea93b653e48","unresolved":false,"context_lines":[{"line_number":258,"context_line":"                                                            version, \u0027shell\u0027)"},{"line_number":259,"context_line":"        except ImportError as e:"},{"line_number":260,"context_line":"            msg \u003d _(\"Invalid client version \u0027%(version)s\u0027. \""},{"line_number":261,"context_line":"                    \"Major part must be one of: \u0027%(major)s\u0027\") % {"},{"line_number":262,"context_line":"                \"version\": version,"},{"line_number":263,"context_line":"                \"major\": \", \".join(self.get_available_major_versions())}"},{"line_number":264,"context_line":"            raise exceptions.UnsupportedVersion("},{"line_number":265,"context_line":"                _(\u0027%(message)s, error was: %(error)s\u0027) %"},{"line_number":266,"context_line":"                {\u0027message\u0027: msg, \u0027error\u0027: e})"}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_a860fca5","line":263,"range":{"start_line":261,"start_character":64,"end_line":263,"end_character":72},"in_reply_to":"fa31d9ce_f0111f53","updated":"2017-02-27 11:08:48.000000000","message":"heh indeed, perhaps wrapping it all with parentheses would allow it to be better indented. But, well... pep8 seems happy with this too","commit_id":"97f82c5a46f63f99129bc3d504928e4c062095dd"}]}