From 3a1280d2921ff4ef22e017ab2d5dbda4be2bbe17 Mon Sep 17 00:00:00 2001 From: Teal Dulcet Date: Wed, 18 Jun 2025 04:19:08 -0700 Subject: [PATCH] Fixed PLW0120 (useless-else-on-loop): `else` clause on loop without a `break` statement; remove the `else` and dedent its contents --- management/backup.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/management/backup.py b/management/backup.py index 900beff8..2b90efd0 100755 --- a/management/backup.py +++ b/management/backup.py @@ -178,10 +178,8 @@ def should_force_full(config, env): if dateutil.parser.parse(bak["date"]) + datetime.timedelta(days=config["min_age_in_days"]*10+1) < datetime.datetime.now(dateutil.tz.tzlocal()): return True return False - else: - # If we got here there are no (full) backups, so make one. - # (I love for/else blocks. Here it's just to show off.) - return True + # If we got here there are no (full) backups, so make one. + return True def get_passphrase(env): # Get the encryption passphrase. secret_key.txt is 2048 random