Simplify.

Author: tobiasraabe

src/_pytask/click.py

@@ -36,66 +36,40 @@
 
 if importlib.metadata.version("click") < "8.2":
     from click.parser import split_opt
-
-    class EnumChoice(Choice):  # type: ignore[type-arg, unused-ignore]
-        """An enum-based choice type.
-
-        The implementation is copied from https://github.com/pallets/click/pull/2210 and
-        related discussion can be found in https://github.com/pallets/click/issues/605.
-
-        In contrast to using :class:`click.Choice`, using this type ensures that the
-        error message does not show the enum members.
-
-        In contrast to the proposed implementation in the PR, this implementation does
-        not use the members than rather the values of the enum.
-
-        """
-
-        def __init__(self, enum_type: type[Enum], case_sensitive: bool = True) -> None:
-            super().__init__(
-                choices=[element.value for element in enum_type],
-                case_sensitive=case_sensitive,
-            )
-            self.enum_type = enum_type
-
-        def convert(
-            self, value: Any, param: Parameter | None, ctx: Context | None
-        ) -> Any:
-            if isinstance(value, Enum):
-                value = value.value
-            value = super().convert(value=value, param=param, ctx=ctx)
-            if value is None:
-                return None
-            return self.enum_type(value)
-
 else:
     from click.parser import (  # type: ignore[attr-defined, no-redef, unused-ignore]
         _split_opt as split_opt,
     )
 
-    class EnumChoice(Choice):  # type: ignore[no-redef, type-arg, unused-ignore]
-        """An enum-based choice type for click >= 8.2.
 
-        Extracts values from enum members to use as valid choices.
+class EnumChoice(Choice):  # type: ignore[type-arg, unused-ignore]
+    """An enum-based choice type.
 
-        """
+    The implementation is copied from https://github.com/pallets/click/pull/2210 and
+    related discussion can be found in https://github.com/pallets/click/issues/605.
 
-        def __init__(self, enum_type: type[Enum], case_sensitive: bool = False) -> None:
-            super().__init__(
-                choices=[element.value for element in enum_type],
-                case_sensitive=case_sensitive,
-            )
-            self.enum_type = enum_type
-
-        def convert(
-            self, value: Any, param: Parameter | None, ctx: Context | None
-        ) -> Any:
-            if isinstance(value, Enum):
-                value = value.value
-            value = super().convert(value=value, param=param, ctx=ctx)
-            if value is None:
-                return None
-            return self.enum_type(value)
+    In contrast to using :class:`click.Choice`, using this type ensures that the
+    error message does not show the enum members.
+
+    In contrast to the proposed implementation in the PR, this implementation does
+    not use the members than rather the values of the enum.
+
+    """
+
+    def __init__(self, enum_type: type[Enum], case_sensitive: bool = True) -> None:
+        super().__init__(
+            choices=[element.value for element in enum_type],
+            case_sensitive=case_sensitive,
+        )
+        self.enum_type = enum_type
+
+    def convert(self, value: Any, param: Parameter | None, ctx: Context | None) -> Any:
+        if isinstance(value, Enum):
+            value = value.value
+        value = super().convert(value=value, param=param, ctx=ctx)
+        if value is None:
+            return None
+        return self.enum_type(value)
 
 
 class _OptionHighlighter(RegexHighlighter):