diff options
author | 2022-12-30 21:03:36 +0200 | |
---|---|---|
committer | 2022-12-30 21:03:36 +0200 | |
commit | 47d21307c44cad87641e20f50f57b3f5d218c0f4 (patch) | |
tree | fcc33f4612f6eac96e615d188591529d5d6d206b | |
parent | fix TestArComp::test_missing_tar when gtar is installed (diff) | |
download | snakeoil-47d21307c44cad87641e20f50f57b3f5d218c0f4.tar.gz snakeoil-47d21307c44cad87641e20f50f57b3f5d218c0f4.tar.bz2 snakeoil-47d21307c44cad87641e20f50f57b3f5d218c0f4.zip |
little modernization of snakeoil.cli
Signed-off-by: Arthur Zamarin <arthurzam@gentoo.org>
-rw-r--r-- | src/snakeoil/cli/arghparse.py | 48 | ||||
-rw-r--r-- | src/snakeoil/cli/input.py | 5 | ||||
-rw-r--r-- | src/snakeoil/cli/tool.py | 2 |
3 files changed, 26 insertions, 29 deletions
diff --git a/src/snakeoil/cli/arghparse.py b/src/snakeoil/cli/arghparse.py index 774699ed..a7a29826 100644 --- a/src/snakeoil/cli/arghparse.py +++ b/src/snakeoil/cli/arghparse.py @@ -148,13 +148,13 @@ class CommaSeparatedValues(argparse._AppendAction): """Split comma-separated values into a list.""" def parse_values(self, values): - items = [] if isinstance(values, str): - items.extend(x for x in values.split(",") if x) + return list(filter(None, values.split(","))) else: + items = [] for value in values: items.extend(x for x in value.split(",") if x) - return items + return items def __call__(self, parser, namespace, values, option_string=None): items = self.parse_values(values) @@ -188,9 +188,9 @@ class CommaSeparatedNegations(argparse._AppendAction): values = [values] for value in values: try: - neg, pos = split_negations(x for x in value.split(",") if x) - except ValueError as e: - raise argparse.ArgumentTypeError(e) + neg, pos = split_negations(filter(None, value.split(","))) + except ValueError as exc: + raise argparse.ArgumentTypeError(exc) disabled.extend(neg) enabled.extend(pos) @@ -236,9 +236,9 @@ class CommaSeparatedElements(argparse._AppendAction): values = [values] for value in values: try: - neg, neu, pos = split_elements(x for x in value.split(",") if x) - except ValueError as e: - raise argparse.ArgumentTypeError(e) + neg, neu, pos = split_elements(filter(None, value.split(","))) + except ValueError as exc: + raise argparse.ArgumentTypeError(exc) disabled.extend(neg) neutral.extend(neu) enabled.extend(pos) @@ -323,7 +323,7 @@ class StoreBool(argparse._StoreAction): return True elif value in ("n", "no", "false", "0"): return False - raise ValueError("value %r must be [y|yes|true|1|n|no|false|0]" % (value,)) + raise ValueError(f"value {value!r} must be [y|yes|true|1|n|no|false|0]") class EnableDebug(argparse._StoreTrueAction): @@ -469,7 +469,7 @@ class Expansion(argparse.Action): args = [x % dvals for x in args] if not action: raise ValueError( - "unable to find option %r for %r" % (option, self.option_strings) + f"unable to find option {option!r} for {self.option_strings!r}" ) if action.type is not None: args = list(map(action.type, args)) @@ -1137,7 +1137,7 @@ class ArgumentParser(OptionalsParser, CsvActionsParser): help="enable/disable color support", docs=""" Toggle colored output support. This can be used to forcibly - enable color support when piping output or other sitations + enable color support when piping output or other situations where stdout is not a tty. """, ) @@ -1273,7 +1273,7 @@ class ArgumentParser(OptionalsParser, CsvActionsParser): namespace.main_func = subcmd_parser.__main_func if unknown_args: - self.error("unrecognized arguments: %s" % " ".join(unknown_args)) + self.error(f"unrecognized arguments: {' '.join(unknown_args)}") # Two runs are required; first, handle any suppression defaults # introduced. Subparsers defaults cannot override the parent parser, as @@ -1302,19 +1302,17 @@ class ArgumentParser(OptionalsParser, CsvActionsParser): try: for attr, delayed in sorted(i, key=lambda val: val[1].priority): delayed(args, attr) - except (TypeError, ValueError) as e: - raise TypeError("failed loading/parsing '%s': %s" % (attr, str(e))) from e + except (TypeError, ValueError) as exc: + raise TypeError(f"failed loading/parsing {attr!r}: {exc}") from exc except argparse.ArgumentError: - e = sys.exc_info()[1] - self.error(str(e)) + exc = sys.exc_info()[1] + self.error(str(exc)) # run final arg validation - final_checks = [ - k for k in args.__dict__.keys() if k.startswith("__final_check__") - ] - for check in final_checks: - functor = args.pop(check) - functor(self, args) + for check in set(vars(args).keys()): + if check.startswith("__final_check__"): + functor = args.pop(check) + functor(self, args) return args @@ -1327,7 +1325,7 @@ class ArgumentParser(OptionalsParser, CsvActionsParser): if self.debug and sys.exc_info() != (None, None, None): # output traceback if any exception is on the stack traceback.print_exc() - self.exit(status, "%s: error: %s\n" % (self.prog, message)) + self.exit(status, f"{self.prog}: error: {message}\n") def bind_main_func(self, functor): """Decorator to set a main function for the parser.""" @@ -1340,7 +1338,7 @@ class ArgumentParser(OptionalsParser, CsvActionsParser): def bind_class(self, obj): if not isinstance(obj, ArgparseCommand): raise ValueError( - "expected obj to be an instance of " "ArgparseCommand; got %r" % (obj,) + f"expected obj to be an instance of ArgparseCommand; got {obj!r}" ) obj.bind_to_parser(self) return self diff --git a/src/snakeoil/cli/input.py b/src/snakeoil/cli/input.py index 9db9ea8c..7e63e332 100644 --- a/src/snakeoil/cli/input.py +++ b/src/snakeoil/cli/input.py @@ -91,11 +91,10 @@ def userquery(prompt, out, err, responses=None, default_answer=None, limit=3): ) ) if not results: - err.write("Sorry, response %r not understood." % (response,)) + err.write(f"Sorry, response {response!r} not understood.") elif len(results) > 1: err.write( - "Response %r is ambiguous (%s)" - % (response, ", ".join(key for key, val in results)) + f"Response {response!r} is ambiguous ({', '.join(key for key, _ in results)})" ) else: return list(results)[0][1][0] diff --git a/src/snakeoil/cli/tool.py b/src/snakeoil/cli/tool.py index d00b3cf2..18b2f89d 100644 --- a/src/snakeoil/cli/tool.py +++ b/src/snakeoil/cli/tool.py @@ -234,5 +234,5 @@ class FormattingHandler(logging.Handler): self.handleError(record) finally: self.out.later_prefix.pop() - for i in range(len(first_prefix)): + for _ in range(len(first_prefix)): self.out.first_prefix.pop() |