)]}'
{"monascaclient/shell.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"9952056259b17549519dc3db1972cdb3486ba82f","unresolved":false,"context_lines":[{"line_number":127,"context_line":"                            help\u003d\u0027Defaults to env[OS_USER_DOMAIN_NAME].\u0027)"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"        parser.add_argument(\u0027--os-project-id\u0027,"},{"line_number":130,"context_line":"                            default\u003dutils.env(\u0027OS_PROJECT_ID\u0027, \u0027OS_TENANT_ID\u0027),"},{"line_number":131,"context_line":"                            help\u003d\u0027Defaults to env[OS_PROJECT_ID].\u0027)"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"        parser.add_argument(\u0027--os_project_id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_62c9f676","line":130,"updated":"2017-04-06 06:21:33.000000000","message":"I think I\u0027d add two extra arguments\n\n  --os-tenant-id\n  --os-tenant-name\n\nand add deprecated to help, or if argparse supports that - directly to add_argument call.","commit_id":"282bb67e84aaa554b87d3f6e335580e779d53018"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"9952056259b17549519dc3db1972cdb3486ba82f","unresolved":false,"context_lines":[{"line_number":333,"context_line":"        for var, oldkey, newkey in [(\u0027os_project_name\u0027, \u0027OS_TENANT_NAME\u0027, \u0027OS_PROJECT_NAME\u0027),"},{"line_number":334,"context_line":"                                        (\u0027os_project_id\u0027, \u0027OS_TENANT_id\u0027, \u0027OS_PROJECT_ID\u0027)]:"},{"line_number":335,"context_line":"            if getattr(args, var) is None and utils.env(oldkey):"},{"line_number":336,"context_line":"                import warnings"},{"line_number":337,"context_line":"                warnings.warn(\"{} is deprecated. use {}\".format(oldkey, newkey),"},{"line_number":338,"context_line":"                            DeprecationWarning)"},{"line_number":339,"context_line":"                setattr(args, var, utils.env(oldkey))"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_02c10a55","line":336,"updated":"2017-04-06 06:21:33.000000000","message":"imports should be located in top of the module.","commit_id":"282bb67e84aaa554b87d3f6e335580e779d53018"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"dd71645ff84d36800fc4371bcf6cf9e03e32eb84","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        super(DeprecatedStore, self).__init__(*args, **kwargs)"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def __call__(self, parser, namespace, values, option_string\u003dNone):"},{"line_number":43,"context_line":"        warnings.warn(\"{} is deprecated\".format(\",\".join(self.option_strings)),"},{"line_number":44,"context_line":"                      DeprecationWarning)"},{"line_number":45,"context_line":"        setattr(namespace, self.dest, values)"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"bff0334d_8f9b6816","line":43,"updated":"2017-04-11 11:44:51.000000000","message":"Shouldn\u0027t this be printed somehow ?","commit_id":"80576cc363cb3ea001fd7183cf958184d2d00a1a"}]}