diff --git a/CHANGELOG.md b/CHANGELOG.md index c5f704a..05b26db 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,10 @@ ## Unreleased +### Internal + +- Apply the Ruff `RET` rules [#92](https://github.com/python-backoff/backoff/pull/92) (from [@edgarrmondragon](https://github.com/edgarrmondragon)) + ## [v2.3.1] - 2025-12-18 ### Fixed diff --git a/backoff/_async.py b/backoff/_async.py index 322c097..9fc36d9 100644 --- a/backoff/_async.py +++ b/backoff/_async.py @@ -9,13 +9,12 @@ def _ensure_coroutine(coro_or_func): if inspect.iscoroutinefunction(coro_or_func): return coro_or_func - else: - @functools.wraps(coro_or_func) - async def f(*args, **kwargs): - return coro_or_func(*args, **kwargs) + @functools.wraps(coro_or_func) + async def f(*args, **kwargs): + return coro_or_func(*args, **kwargs) - return f + return f def _ensure_coroutines(coros_or_funcs): @@ -108,9 +107,8 @@ async def retry(*args, **kwargs): # await asyncio.sleep(seconds) continue - else: - await _call_handlers(on_success, **details, value=ret) - break + await _call_handlers(on_success, **details, value=ret) + break return ret diff --git a/backoff/_sync.py b/backoff/_sync.py index 7cafae2..e06b1f1 100644 --- a/backoff/_sync.py +++ b/backoff/_sync.py @@ -70,9 +70,8 @@ def retry(*args, **kwargs): time.sleep(seconds) continue - else: - _call_handlers(on_success, **details, value=ret) - break + _call_handlers(on_success, **details, value=ret) + break return ret diff --git a/docs/examples.md b/docs/examples.md index 72ba5fb..5b3c6e0 100644 --- a/docs/examples.md +++ b/docs/examples.md @@ -73,8 +73,7 @@ from sqlalchemy.exc import OperationalError, TimeoutError ) def connect_to_database(connection_string): engine = sqlalchemy.create_engine(connection_string) - connection = engine.connect() - return connection + return engine.connect() ``` ### Transaction Retry with Deadlock Handling diff --git a/pyproject.toml b/pyproject.toml index d1ff679..bf74635 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -183,6 +183,7 @@ docstring-code-line-length = 20 [tool.ruff.lint] extend-select = [ + "RET", # flake8-return "SIM", # flake8-simplify "UP", # pyupgrade ]