mypy strict, ruff improvement

arq/connections.py

@@ -4,7 +4,7 @@
 from dataclasses import dataclass
 from datetime import datetime, timedelta
 from operator import attrgetter
-from typing import TYPE_CHECKING, Any, Callable, List, Optional, Tuple, Union
+from typing import TYPE_CHECKING, Any, Callable, List, Optional, Tuple, Union, cast
 from urllib.parse import parse_qs, urlparse
 from uuid import uuid4
 
@@ -237,7 +237,8 @@ def pool_factory(*args: Any, **kwargs: Any) -> ArqRedis:
                 ssl=settings.ssl,
                 **kwargs,
             )
-            return client.master_for(settings.sentinel_master, redis_class=ArqRedis)
+            redis = client.master_for(settings.sentinel_master, redis_class=ArqRedis)
+            return cast(ArqRedis, redis)
 
     else:
         pool_factory = functools.partial(

pyproject.toml

@@ -57,11 +57,7 @@ Changelog = 'https://github.com/samuelcolvin/arq/releases'
 
 [tool.pytest.ini_options]
 testpaths = 'tests'
-filterwarnings = [
-    'error',
-#    'ignore:pkg_resources is deprecated as an API.*:DeprecationWarning',
-#    'ignore:Deprecated call to.*:DeprecationWarning',
-]
+filterwarnings = ['error']
 asyncio_mode = 'auto'
 timeout = 10
 
@@ -84,34 +80,12 @@ exclude_lines = [
 line-length = 120
 
 [tool.ruff.lint]
-extend-select = ['Q', 'RUF100', 'C90', 'I']
-extend-ignore = [
-    'E721',  # using type() instead of isinstance() - we use this in tests
-]
+extend-select = ['Q', 'RUF100', 'C90', 'UP', 'I']
 flake8-quotes = {inline-quotes = 'single', multiline-quotes = 'double'}
 mccabe = { max-complexity = 13 }
 
 [tool.ruff.format]
 quote-style = 'single'
 
 [tool.mypy]
-show_error_codes = true
-follow_imports = 'silent'
-strict_optional = true
-warn_redundant_casts = true
-warn_unused_ignores = true
-disallow_any_generics = true
-check_untyped_defs = true
-no_implicit_reexport = true
-warn_unused_configs = true
-disallow_subclassing_any = true
-disallow_incomplete_defs = true
-disallow_untyped_decorators = true
-disallow_untyped_calls = true
-
-# for strict mypy: (this is the tricky one :-))
-disallow_untyped_defs = true
-
-# remaining arguments from `mypy --strict` which cause errors
-#no_implicit_optional = true
-#warn_return_any = true
+strict = true