chore: Allow combining non base-derived objects for retries

releasenotes/notes/allow-retry-callables-ba921a2b57229540.yaml

@@ -0,0 +1,7 @@
+---
+features:
+  - |
+    Allow for callables to be combined as retry values. This will only
+    work when used combined with their corresponding implementation
+    retry objects, e.g. only async functions will work when used together
+    with async retry strategies.

tenacity/asyncio/retry.py

@@ -104,7 +104,7 @@ def __init__(self, *retries: typing.Union[retry_base, async_retry_base]) -> None
     async def __call__(self, retry_state: "RetryCallState") -> bool:  # type: ignore[override]
         result = False
         for r in self.retries:
-            result = result or await _utils.wrap_to_async_func(r)(retry_state)
+            result = result or (await _utils.wrap_to_async_func(r)(retry_state) is True)
             if result:
                 break
         return result
@@ -119,7 +119,9 @@ def __init__(self, *retries: typing.Union[retry_base, async_retry_base]) -> None
     async def __call__(self, retry_state: "RetryCallState") -> bool:  # type: ignore[override]
         result = True
         for r in self.retries:
-            result = result and await _utils.wrap_to_async_func(r)(retry_state)
+            result = result and (
+                await _utils.wrap_to_async_func(r)(retry_state) is True
+            )
             if not result:
                 break
         return result

tenacity/retry.py

@@ -30,15 +30,29 @@ def __call__(self, retry_state: "RetryCallState") -> bool:
         pass
 
     def __and__(self, other: "retry_base") -> "retry_all":
-        return other.__rand__(self)
+        if isinstance(other, retry_base):
+            # Delegate to the other object to allow for specific
+            # implementations, such as asyncio
+            return other.__rand__(self)
+        return retry_all(other, self)
 
     def __rand__(self, other: "retry_base") -> "retry_all":
+        # This is automatically invoked for inheriting classes,
+        # so it helps to keep the abstraction and delegate specific
+        # implementations, such as asyncio
         return retry_all(other, self)
 
     def __or__(self, other: "retry_base") -> "retry_any":
-        return other.__ror__(self)
+        if isinstance(other, retry_base):
+            # Delegate to the other object to allow for specific
+            # implementations, such as asyncio
+            return other.__ror__(self)
+        return retry_any(other, self)
 
     def __ror__(self, other: "retry_base") -> "retry_any":
+        # This is automatically invoked for inheriting classes,
+        # so it helps to keep the abstraction and delegate specific
+        # implementations, such as asyncio
         return retry_any(other, self)
 
 
@@ -269,7 +283,7 @@ def __init__(self, *retries: retry_base) -> None:
         self.retries = retries
 
     def __call__(self, retry_state: "RetryCallState") -> bool:
-        return any(r(retry_state) for r in self.retries)
+        return any(r(retry_state) is True for r in self.retries)
 
 
 class retry_all(retry_base):
@@ -279,4 +293,4 @@ def __init__(self, *retries: retry_base) -> None:
         self.retries = retries
 
     def __call__(self, retry_state: "RetryCallState") -> bool:
-        return all(r(retry_state) for r in self.retries)
+        return all(r(retry_state) is True for r in self.retries)