Merge pull request #691 from Avasam/add-typed-libraries-to-type-checking

Re-enable mypy and add libraries to type checking

keyring/backend.py

@@ -114,6 +114,7 @@ def name(cls) -> str:
         """
         parent, sep, mod_name = cls.__module__.rpartition('.')
         mod_name = mod_name.replace('_', ' ')
+        # mypy doesn't see `cls` is `type[Self]`, might be fixable in jaraco.classes
         return ' '.join([mod_name, cls.__name__])  # type: ignore[attr-defined]
 
     def __str__(self) -> str:

keyring/cli.py

@@ -13,15 +13,24 @@
     core,
     credentials,
     delete_password,
+    get_credential,
     get_password,
     set_keyring,
     set_password,
-    get_credential,
 )
 from .util import platform_
 
 
 class CommandLineTool:
+    # Attributes set dynamically by the ArgumentParser
+    keyring_path: str | None
+    keyring_backend: str | None
+    get_mode: str
+    output_format: str
+    operation: str
+    service: str
+    username: str
+
     def __init__(self):
         self.parser = argparse.ArgumentParser()
         self.parser.add_argument(
@@ -133,10 +142,10 @@ def _emit_plain(self, credential: credentials.Credential):
         print(credential.password)
 
     def _get_creds(self) -> credentials.Credential | None:
-        return get_credential(self.service, self.username)  # type: ignore[attr-defined]
+        return get_credential(self.service, self.username)
 
     def _get_password(self) -> credentials.Credential | None:
-        password = get_password(self.service, self.username)  # type: ignore[attr-defined]
+        password = get_password(self.service, self.username)
         return (
             credentials.AnonymousCredential(password) if password is not None else None
         )

keyring/compat/py312.py

@@ -3,7 +3,7 @@
 __all__ = ['metadata']
 
 
-if sys.version_info > (3, 12):
+if sys.version_info >= (3, 12):
     import importlib.metadata as metadata
 else:
-    import importlib_metadata as metadata  # type: ignore[import-not-found, unused-ignore]
+    import importlib_metadata as metadata

keyring/core.py

@@ -106,7 +106,7 @@ def _detect_backend(limit: typing.Optional[LimitCallable] = None):
         or load_config()
         or max(
             # all keyrings passing the limit filter
-            filter(limit, backend.get_all_keyring()),  # type: ignore[arg-type]  # 659
+            filter(limit, backend.get_all_keyring()),  # type: ignore[arg-type] #659
             default=fail.Keyring(),
             key=backend.by_priority,
         )

mypy.ini

@@ -14,6 +14,6 @@ disable_error_code =
 	# Disable due to many false positives
 	overload-overlap,
 
-# TODO: associate issues with these exclusions
-[mypy-win32ctypes.*,win32cred.*,pywintypes,secretstorage.*,dbus.*,gi.*,shtab]
+# TODO: Open upstream issues requesting typing
+[mypy-win32ctypes.*,secretstorage.*,dbus.*]
 ignore_missing_imports = True

pyproject.toml

@@ -76,6 +76,9 @@ type = [
 	"pytest-mypy",
 
 	# local
+	"pygobject-stubs",
+	"shtab", # Optional install for completion
+	"types-pywin32",
 ]
 
 completion = ["shtab >= 1.1.0"]
@@ -96,7 +99,3 @@ keyring = "keyring.cli:main"
 
 
 [tool.setuptools_scm]
-
-
-[tool.pytest-enabler.mypy]
-# Disabled due to jaraco/skeleton#143