diff options
| author | Eric N. Vander Weele <ericvw@gmail.com> | 2019-08-30 15:17:16 -0400 |
|---|---|---|
| committer | Eric N. Vander Weele <ericvw@gmail.com> | 2019-08-31 00:10:40 -0400 |
| commit | ec8fcfc8f8fb3a4dd69e3679c5076baf01481e06 (patch) | |
| tree | 1e2b3cb9b52ff8aa318ec8e4aee35f4c2770b19b | |
| parent | eee53d9440743a81a5e16b5f75da39832ee3e5ff (diff) | |
| download | flake8-ec8fcfc8f8fb3a4dd69e3679c5076baf01481e06.tar.gz | |
Add typing to `OptionManager.parse_args()`
Note that the `assert` is necessary to "cast" `self.parser` since it is
specified as a `Union`.
| -rw-r--r-- | src/flake8/options/manager.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/flake8/options/manager.py b/src/flake8/options/manager.py index a5e5906..3aea4ad 100644 --- a/src/flake8/options/manager.py +++ b/src/flake8/options/manager.py @@ -425,13 +425,21 @@ class OptionManager(object): plugin_version_format ) - def parse_args(self, args=None, values=None): + def parse_args( + self, + args=None, # type: Optional[List[str]] + values=None, # type: Optional[argparse.Namespace] + ): + # type: (...) -> Tuple[argparse.Namespace, List[str]] """Proxy to calling the OptionParser's parse_args method.""" self.generate_epilog() self.update_version_string() - args = self.parser.parse_args(args, values) + assert isinstance( # nosec (for bandit) + self.parser, argparse.ArgumentParser + ), self.parser + parsed_args = self.parser.parse_args(args, values) # TODO: refactor callers to not need this - return args, args.filenames + return parsed_args, parsed_args.filenames def parse_known_args(self, args=None): # type: (Optional[List[str]]) -> Tuple[argparse.Namespace, List[str]] |
