refactor: remove else part of for ... else in @Retry

IATkachenko · IATkachenko · commit 9684312e3060 · 2022-09-18T20:02:29.000+03:00
Actually we don't need it, because we have `return` if target function was run without exceptions.
diff --git a/tion_btle/tion.py b/tion_btle/tion.py
@@ -37,14 +37,15 @@ async def wrapper(*args, **kwargs):
                     pass
                 if delay > 0:
                     await asyncio.sleep(delay)
-            else:
-                _LOGGER.critical("Retry limit (%d) exceeded for %s(%s, %s)", retries, f.__name__, args, kwargs)
-                if _LOGGER.level > logging.INFO and last_info_exception is not None:
-                    _LOGGER.critical(f"Last exception was {last_info_exception}")
-                elif _LOGGER.level > logging.WARNING and last_warning_exception is not None:
-                    _LOGGER.critical(f"Last exception was {last_warning_exception}")
 
-                raise MaxTriesExceededError
+            _LOGGER.critical("Retry limit (%d) exceeded for %s(%s, %s)", retries, f.__name__, args, kwargs)
+            if _LOGGER.level > logging.INFO and last_info_exception is not None:
+                _LOGGER.critical(f"Last exception was {last_info_exception}")
+            elif _LOGGER.level > logging.WARNING and last_warning_exception is not None:
+                _LOGGER.critical(f"Last exception was {last_warning_exception}")
+
+            raise MaxTriesExceededError
+
         return wrapper
     return decor
 
