summary refs log tree commit diff
path: root/scripts-dev/check_pydantic_models.py
diff options
context:
space:
mode:
authorDavid Robertson <davidr@element.io>2022-09-30 16:34:47 +0100
committerGitHub <noreply@github.com>2022-09-30 16:34:47 +0100
commit6d543d6d9f56e39199b7e460d0081b02d61f12be (patch)
treef9e3193798794dc1c8e085bb6e654574f27e166d /scripts-dev/check_pydantic_models.py
parentEnable dependabot updates (#13976) (diff)
downloadsynapse-6d543d6d9f56e39199b7e460d0081b02d61f12be.tar.xz
Update mypy and mypy-zope (#13925)
* Update mypy and mypy-zope

* Unignore assigning to LogRecord attributes

Presumably https://github.com/python/typeshed/pull/8064 makes this ok

Cherry-picked from #13521

* Remove unused ignores due to mypy ParamSpec fixes

https://github.com/python/mypy/pull/12668

Cherry-picked from #13521

* Remove additional unused ignores

* Fix new mypy complaints related to `assertGreater`

Presumably due to https://github.com/python/typeshed/pull/8077

* Changelog

* Reword changelog

Co-authored-by: Patrick Cloke <clokep@users.noreply.github.com>

Co-authored-by: Patrick Cloke <clokep@users.noreply.github.com>
Diffstat (limited to 'scripts-dev/check_pydantic_models.py')
-rwxr-xr-xscripts-dev/check_pydantic_models.py5
1 files changed, 2 insertions, 3 deletions
diff --git a/scripts-dev/check_pydantic_models.py b/scripts-dev/check_pydantic_models.py
index d0fb811bdb..9f2b7ded5b 100755
--- a/scripts-dev/check_pydantic_models.py
+++ b/scripts-dev/check_pydantic_models.py
@@ -88,10 +88,9 @@ def make_wrapper(factory: Callable[P, R]) -> Callable[P, R]:
 
     @functools.wraps(factory)
     def wrapper(*args: P.args, **kwargs: P.kwargs) -> R:
-        # type-ignore: should be redundant once we can use https://github.com/python/mypy/pull/12668
-        if "strict" not in kwargs:  # type: ignore[attr-defined]
+        if "strict" not in kwargs:
             raise MissingStrictInConstrainedTypeException(factory.__name__)
-        if not kwargs["strict"]:  # type: ignore[index]
+        if not kwargs["strict"]:
             raise MissingStrictInConstrainedTypeException(factory.__name__)
         return factory(*args, **kwargs)