Don't store a RichHandler object in the argparse namespace (#3394) (#…

…3398)
getpelican · Sep 28, 2024 · 0cb445c · 0cb445c
2 parents 84db21c + 6c6b940
commit 0cb445c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/pelican/__init__.py b/pelican/__init__.py
@@ -310,6 +310,9 @@ def __call__(self, parser, namespace, values, option_string=None):
         setattr(namespace, self.dest, overrides)
 
 
+LOG_HANDLERS = {"plain": None, "rich": DEFAULT_LOG_HANDLER}
+
+
 def parse_arguments(argv=None):
     parser = argparse.ArgumentParser(
         description="A tool to generate a static blog, "
@@ -448,7 +451,6 @@ def parse_arguments(argv=None):
         ),
     )
 
-    LOG_HANDLERS = {"plain": None, "rich": DEFAULT_LOG_HANDLER}
     parser.add_argument(
         "--log-handler",
         default="rich",
@@ -513,8 +515,6 @@ def parse_arguments(argv=None):
     if args.bind is not None and not args.listen:
         logger.warning("--bind without --listen has no effect")
 
-    args.log_handler = LOG_HANDLERS[args.log_handler]
-
     return args
 
 
@@ -637,7 +637,7 @@ def main(argv=None):
         level=args.verbosity,
         fatal=args.fatal,
         name=__name__,
-        handler=args.log_handler,
+        handler=LOG_HANDLERS[args.log_handler],
         logs_dedup_min_level=logs_dedup_min_level,
     )