diff --git a/backoff/_common.py b/backoff/_common.py index 2b2e54e..468ab32 100644 --- a/backoff/_common.py +++ b/backoff/_common.py @@ -65,14 +65,14 @@ def _prepare_logger(logger): # Configure handler list with user specified handler and optionally # with a default handler bound to the specified logger. def _config_handlers( - user_handlers, *, default_handler=None, logger=None, log_level=None + user_handlers, *, default_handler=None, logger=None, log_level=None, ): handlers = [] if logger is not None: assert log_level is not None, "Log level is not specified" # bind the specified logger to the default log handler log_handler = functools.partial( - default_handler, logger=logger, log_level=log_level + default_handler, logger=logger, log_level=log_level, ) handlers.append(log_handler) diff --git a/backoff/_decorator.py b/backoff/_decorator.py index ca5d0ff..5aeb284 100644 --- a/backoff/_decorator.py +++ b/backoff/_decorator.py @@ -8,7 +8,7 @@ _prepare_logger, _config_handlers, _log_backoff, - _log_giveup + _log_giveup, ) from backoff._jitter import full_jitter from backoff import _async, _sync @@ -89,13 +89,13 @@ def decorate(target): on_backoff, default_handler=_log_backoff, logger=logger, - log_level=backoff_log_level + log_level=backoff_log_level, ) on_giveup = _config_handlers( on_giveup, default_handler=_log_giveup, logger=logger, - log_level=giveup_log_level + log_level=giveup_log_level, ) if inspect.iscoroutinefunction(target): @@ -113,7 +113,7 @@ def decorate(target): on_success=on_success, on_backoff=on_backoff, on_giveup=on_giveup, - wait_gen_kwargs=wait_gen_kwargs + wait_gen_kwargs=wait_gen_kwargs, ) # Return a function which decorates a target with a retry loop. @@ -215,7 +215,7 @@ def decorate(target): on_backoff=on_backoff, on_giveup=on_giveup, raise_on_giveup=raise_on_giveup, - wait_gen_kwargs=wait_gen_kwargs + wait_gen_kwargs=wait_gen_kwargs, ) # Return a function which decorates a target with a retry loop. diff --git a/backoff/_wait_gen.py b/backoff/_wait_gen.py index 33b37b8..e6bed09 100644 --- a/backoff/_wait_gen.py +++ b/backoff/_wait_gen.py @@ -8,7 +8,7 @@ def expo( base: float = 2, factor: float = 1, - max_value: Optional[float] = None + max_value: Optional[float] = None, ) -> Generator[float, Any, None]: """Generator for exponential decay. @@ -35,7 +35,7 @@ def expo( def decay( initial_value: float = 1, decay_factor: float = 1, - min_value: Optional[float] = None + min_value: Optional[float] = None, ) -> Generator[float, Any, None]: """Generator for exponential decay[1]: @@ -83,7 +83,7 @@ def fibo(max_value: Optional[int] = None) -> Generator[int, None, None]: def constant( - interval: Union[int, Iterable[float]] = 1 + interval: Union[int, Iterable[float]] = 1, ) -> Generator[float, None, None]: """Generator for constant intervals. @@ -104,7 +104,7 @@ def constant( def runtime( *, - value: Callable[[Any], float] + value: Callable[[Any], float], ) -> Generator[float, None, None]: """Generator that is based on parsing the return value or thrown exception of the decorated method diff --git a/backoff/types.py b/backoff/types.py index 25f20a4..fc280aa 100644 --- a/backoff/types.py +++ b/backoff/types.py @@ -2,5 +2,5 @@ from ._typing import Details __all__ = [ - 'Details' + 'Details', ] diff --git a/pyproject.toml b/pyproject.toml index 2258bc7..4ee8fb4 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -120,6 +120,7 @@ description = "format code" dependency_groups = [ "lint" ] commands = [ [ "ruff", "check", "--fix", { replace = "posargs", default = [ "backoff", "tests" ], extend = true } ], + # [ "ruff", "format", { replace = "posargs", default = [ "backoff", "tests" ], extend = true } ], ] [tool.tox.env.lint] @@ -179,6 +180,11 @@ source = [ "backoff", ] +[tool.ruff.lint] +extend-select = [ + "COM", # flake8-commas +] + [build-system] requires = ["hatchling"] build-backend = "hatchling.build" diff --git a/tests/test_backoff.py b/tests/test_backoff.py index 7d46269..d68cc4e 100644 --- a/tests/test_backoff.py +++ b/tests/test_backoff.py @@ -49,7 +49,7 @@ def test_on_predicate_max_time(monkeypatch): 10.000005, 9, 1, - 0 + 0, ] def monotonic(): @@ -80,7 +80,7 @@ def test_on_predicate_max_time_callable(monkeypatch): 10.000005, 9, 1, - 0 + 0, ] def monotonic(): @@ -840,7 +840,7 @@ def key_error(): def _on_exception_factory( - backoff_log_level, giveup_log_level, max_tries + backoff_log_level, giveup_log_level, max_tries, ): @backoff.on_exception( backoff.expo, @@ -860,7 +860,7 @@ def func(): def _on_predicate_factory( - backoff_log_level, giveup_log_level, max_tries + backoff_log_level, giveup_log_level, max_tries, ): @backoff.on_predicate( backoff.expo, @@ -889,17 +889,17 @@ def func(): repeat=2, ) for factory in (_on_predicate_factory, _on_exception_factory) - ) + ), ) def test_event_log_levels( - caplog, func_factory, backoff_log_level, giveup_log_level + caplog, func_factory, backoff_log_level, giveup_log_level, ): max_tries = 3 func = func_factory(backoff_log_level, giveup_log_level, max_tries) with unittest.mock.patch('time.sleep', return_value=None): with caplog.at_level( - min(backoff_log_level, giveup_log_level), logger="backoff" + min(backoff_log_level, giveup_log_level), logger="backoff", ): func() diff --git a/tests/test_typing.py b/tests/test_typing.py index 7f53459..ea3b27c 100644 --- a/tests/test_typing.py +++ b/tests/test_typing.py @@ -18,7 +18,7 @@ def foo(): backoff.constant, ValueError, interval=1, - max_tries=3 + max_tries=3, ) def bar(): raise ValueError()