chore(cli): remove unused PERF verbosity level (#245)

* chore(cli): remove unused PERF verbosity level

As titled.

* fix(tests): remove perf in tests

Author: jeertmans

CHANGELOG.md

@@ -52,3 +52,5 @@ In an effort to better document changes, this CHANGELOG document is now created.
 - Removed the following options from `manim-slides present`:
   `--resolution`, `--record-to`, `--resize-mode`, and `--background-color`.
   [#243](https://github.com/jeertmans/manim-slides/pull/243)
+- Removed `PERF` verbosity level because not used anymore.
+  [#245](https://github.com/jeertmans/manim-slides/pull/245)

manim_slides/commons.py

@@ -54,10 +54,10 @@ def callback(ctx: Context, param: Parameter, value: str) -> None:
         "-v",
         "--verbosity",
         type=click.Choice(
-            ["PERF", "DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"],
+            ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"],
             case_sensitive=False,
         ),
-        help="Verbosity of CLI output. PERF will log performances (timing) information.",
+        help="Verbosity of CLI output.",
         default=None,
         expose_value=False,
         envvar="MANIM_SLIDES_VERBOSITY",

manim_slides/logger.py

@@ -7,7 +7,6 @@
 
 from rich.console import Console
 from rich.logging import RichHandler
-from rich.theme import Theme
 
 __all__ = ["logger", "make_logger"]
 
@@ -36,9 +35,8 @@ def make_logger() -> logging.Logger:
     RichHandler.KEYWORDS = HIGHLIGHTED_KEYWORDS
     rich_handler = RichHandler(
         show_time=True,
-        console=Console(theme=Theme({"logging.level.perf": "magenta"})),
+        console=Console(),
     )
-    logging.addLevelName(5, "PERF")
     logger = logging.getLogger("manim-slides")
     logger.setLevel(logging.getLogger("manim").level)
     logger.addHandler(rich_handler)

tests/conftest.py

@@ -8,7 +8,7 @@
 from manim_slides.config import PresentationConfig
 from manim_slides.logger import make_logger
 
-_ = make_logger()  # This is run so that "PERF" level is created
+_ = make_logger()  # This is run so that logger is created
 
 
 @pytest.fixture

tests/test_commons.py

@@ -95,7 +95,7 @@ def main(folder: Path) -> None:
 
 @pytest.mark.parametrize(
     ("verbosity",),
-    [("PeRF",), ("DEBUG",), ("info",), ("waRNING",), ("eRRor",), ("CrItIcAl",)],
+    [("DEBUG",), ("info",), ("waRNING",), ("eRRor",), ("CrItIcAl",)],
 )
 def test_valid_verbosity_option(verbosity: str) -> None:
     @click.command()