style: better type

Author: bearomorphism

commitizen/cli.py

@@ -8,7 +8,6 @@
 from functools import partial
 from pathlib import Path
 from types import TracebackType
-from typing import Any
 
 import argcomplete
 from decli import cli
@@ -48,7 +47,7 @@ def __call__(
         self,
         parser: argparse.ArgumentParser,
         namespace: argparse.Namespace,
-        kwarg: str | Sequence[Any] | None,
+        kwarg: str | Sequence[object] | None,
         option_string: str | None = None,
     ) -> None:
         if not isinstance(kwarg, str):

commitizen/providers/base_provider.py

@@ -2,6 +2,7 @@
 
 import json
 from abc import ABC, abstractmethod
+from collections.abc import Mapping
 from pathlib import Path
 from typing import Any, ClassVar
 
@@ -63,8 +64,8 @@ def set_version(self, version: str) -> None:
         self.set(document, version)
         self.file.write_text(json.dumps(document, indent=self.indent) + "\n")
 
-    def get(self, document: dict[str, Any]) -> str:
-        return document["version"]  # type: ignore
+    def get(self, document: Mapping[str, str]) -> str:
+        return document["version"]
 
     def set(self, document: dict[str, Any], version: str) -> None:
         document["version"] = version

commitizen/providers/npm_provider.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import json
+from collections.abc import Mapping
 from pathlib import Path
 from typing import Any, ClassVar
 
@@ -58,8 +59,8 @@ def set_version(self, version: str) -> None:
                 json.dumps(shrinkwrap_document, indent=self.indent) + "\n"
             )
 
-    def get_package_version(self, document: dict[str, Any]) -> str:
-        return document["version"]  # type: ignore
+    def get_package_version(self, document: Mapping[str, str]) -> str:
+        return document["version"]
 
     def set_package_version(
         self, document: dict[str, Any], version: str