diff options
author | Florian Bruhin <me@the-compiler.org> | 2021-12-03 11:58:41 +0100 |
---|---|---|
committer | Florian Bruhin <me@the-compiler.org> | 2021-12-03 12:04:59 +0100 |
commit | 4435ef928796fa744be5452924a103bb2e58ee8e (patch) | |
tree | 73f99cf693fa492ce22e228d16fc1a7eff540e81 /qutebrowser/commands/command.py | |
parent | b84d68e8ee22cb1679799c9ca73397e81110ea34 (diff) | |
download | qutebrowser-pylint-2-12.tar.gz qutebrowser-pylint-2-12.zip |
pylint: Enable else-if-usedpylint-2-12
Diffstat (limited to 'qutebrowser/commands/command.py')
-rw-r--r-- | qutebrowser/commands/command.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/qutebrowser/commands/command.py b/qutebrowser/commands/command.py index 653f551ff..eee5b7cde 100644 --- a/qutebrowser/commands/command.py +++ b/qutebrowser/commands/command.py @@ -272,11 +272,10 @@ class Command: if is_bool: kwargs['action'] = 'store_true' + elif arg_info.metavar is not None: + kwargs['metavar'] = arg_info.metavar else: - if arg_info.metavar is not None: - kwargs['metavar'] = arg_info.metavar - else: - kwargs['metavar'] = argparser.arg_name(param.name) + kwargs['metavar'] = argparser.arg_name(param.name) if param.kind == inspect.Parameter.VAR_POSITIONAL: kwargs['nargs'] = '*' if self._star_args_optional else '+' @@ -320,9 +319,8 @@ class Command: self.opt_args[param.name] = long_flag, short_flag if not is_bool: self.flags_with_args += [short_flag, long_flag] - else: - if not arg_info.hide: - self.pos_args.append((param.name, name)) + elif not arg_info.hide: + self.pos_args.append((param.name, name)) return args def _get_type(self, param): |