)]}'
{"magnumclient/shell.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"19e4f4b7866bf0e4804fb956a8c4a6ce38b645c7","unresolved":false,"context_lines":[{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        try:"},{"line_number":339,"context_line":"            actions_module \u003d {"},{"line_number":340,"context_line":"                \u00271\u0027: shell_v1,"},{"line_number":341,"context_line":"            }[version]"},{"line_number":342,"context_line":"        except KeyError:"},{"line_number":343,"context_line":"            actions_module \u003d shell_v1"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_2992454c","line":340,"updated":"2015-11-13 22:26:36.000000000","message":"I think it is more readable to list all the modules here:\n\nactions_module \u003d {\n    \u00271\u0027: [baymodels_shell, bays_shell, ...],\n}[version]","commit_id":"37c04e678bbbd8853f31fdca859d81a2726de726"},{"author":{"_account_id":12385,"name":"Motohiro/Yuanying Otsuka","email":"yuanying@fraction.jp","username":"yuanying"},"change_message_id":"dfae58f4fa8f855d45022aee6d150e46cbf55167","unresolved":false,"context_lines":[{"line_number":337,"context_line":""},{"line_number":338,"context_line":"        try:"},{"line_number":339,"context_line":"            actions_module \u003d {"},{"line_number":340,"context_line":"                \u00271\u0027: shell_v1,"},{"line_number":341,"context_line":"            }[version]"},{"line_number":342,"context_line":"        except KeyError:"},{"line_number":343,"context_line":"            actions_module \u003d shell_v1"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba8a016a_82c1dfe6","line":340,"in_reply_to":"da85f559_2992454c","updated":"2015-11-17 00:14:38.000000000","message":"+1, I\u0027ll update soon.","commit_id":"37c04e678bbbd8853f31fdca859d81a2726de726"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"19e4f4b7866bf0e4804fb956a8c4a6ce38b645c7","unresolved":false,"context_lines":[{"line_number":343,"context_line":"            actions_module \u003d shell_v1"},{"line_number":344,"context_line":""},{"line_number":345,"context_line":"        actions_module.enhance_parser(parser, subparsers, self.subcommands)"},{"line_number":346,"context_line":"        utils.define_commands_from_module(subparsers, self, self.subcommands)"},{"line_number":347,"context_line":"        self._find_actions(subparsers, self)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"        self._add_bash_completion_subparser(subparsers)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da85f559_c934d94b","line":346,"updated":"2015-11-13 22:26:36.000000000","message":"If you list the modules (as suggested above), the code can be clean up a little:\n\nfor actions_module in actions_modules:\n    self._find_actions(subparsers, actions_module)\n....","commit_id":"37c04e678bbbd8853f31fdca859d81a2726de726"}]}