get_help_record not abstract, return None instead of pass so linter d…

…oesn't complain, add abc allowed import.
pallets · Nov 9, 2024 · c40f6d3 · c40f6d3
1 parent 3cc11cc
commit c40f6d3
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 17 deletions.
diff --git a/src/click/core.py b/src/click/core.py
@@ -2349,8 +2349,8 @@ def handle_parse_result(
 
         return value, args
 
-    @abstractmethod
-    def get_help_record(self, ctx: Context) -> tuple[str, str] | None: ...
+    def get_help_record(self, ctx: Context) -> tuple[str, str] | None:
+        return None
 
     def get_usage_pieces(self, ctx: Context) -> list[str]:
         return []

diff --git a/tests/test_imports.py b/tests/test_imports.py
@@ -27,28 +27,29 @@ def tracking_import(module, locals=None, globals=None, fromlist=None,
 
 ALLOWED_IMPORTS = {
     "__future__",
-    "weakref",
-    "os",
-    "struct",
-    "collections",
+    "abc",
+    "codecs",
     "collections.abc",
-    "sys",
+    "collections",
     "contextlib",
+    "datetime",
+    "enum",
+    "errno",
+    "fcntl",
     "functools",
-    "stat",
-    "re",
-    "codecs",
+    "gettext",
     "inspect",
-    "itertools",
     "io",
+    "itertools",
+    "os",
+    "re",
+    "stat",
+    "struct",
+    "sys",
     "threading",
-    "errno",
-    "fcntl",
-    "datetime",
-    "enum",
-    "typing",
     "types",
-    "gettext",
+    "typing",
+    "weakref",
 }
 
 if WIN: