bump mypy to fix a bunch of issues

coveooss · Mar 21, 2024 · 833424f · 833424f
1 parent 15d339f
commit 833424f
Show file tree

Hide file tree

Showing 29 changed files with 425 additions and 365 deletions.
diff --git a/coveo-arnparse/poetry.lock b/coveo-arnparse/poetry.lock
diff --git a/coveo-arnparse/pyproject.toml b/coveo-arnparse/pyproject.toml
@@ -17,7 +17,7 @@ bandit = "*"
 black = "*"
 coveo-testing = { path = "../coveo-testing", develop = true }
 flake8 = "*"
-mypy = "0.950"
+mypy = "1.9.0"
 pytest = "*"
 
 

diff --git a/coveo-example-library/poetry.lock b/coveo-example-library/poetry.lock
diff --git a/coveo-example-library/pyproject.toml b/coveo-example-library/pyproject.toml
@@ -16,7 +16,7 @@ python = ">=3.8"
 bandit = "*"
 black = "*"
 coveo-testing = { path = "../coveo-testing", develop = true }
-mypy = "0.950"
+mypy = "1.9.0"
 pytest = "*"
 
 [tool.stew.ci]

diff --git a/coveo-functools/coveo_functools/flex/decorator.py b/coveo-functools/coveo_functools/flex/decorator.py
@@ -114,7 +114,7 @@ def _generate_callable_wrapper(fn: RealFunction, errors: ErrorBehavior) -> Wrapp
     """Class decorators"""
 
     @functools.wraps(fn)
-    def wrapper(*args: Any, **kwargs: Any) -> T:
+    def wrapper(*args: Any, **kwargs: Any) -> T:  # type: ignore[type-var]
         value: T = fn(*args, **convert_kwargs_for_unpacking(kwargs, hint=fn, errors=errors))
         if hasattr(value, "__dict__"):
             value.__dict__[RAW_KEY] = kwargs

diff --git a/coveo-functools/coveo_functools/flex/deserializer.py b/coveo-functools/coveo_functools/flex/deserializer.py
@@ -61,7 +61,7 @@ def convert_kwargs_for_unpacking(
 
     # convert the values so they match the additional metadata if available, else fn's annotations.
     converted_kwargs = {}
-    for arg_name, arg_hint in {**find_annotations(fn), **additional_metadata}.items():  # type: ignore[arg-type]
+    for arg_name, arg_hint in {**find_annotations(fn), **additional_metadata}.items():
         if arg_name not in mapped_kwargs:
             continue  # this may be ok, for instance if the target argument has a default