From f54a5458f6db668a5ff4d6395d792e00d20999e7 Mon Sep 17 00:00:00 2001 From: Rodos Date: Wed, 18 Feb 2026 20:10:48 +1100 Subject: [PATCH] Fix empty repository crash due to None timestamp comparison (#489) Empty repositories have None for pushed_at/updated_at, causing a TypeError when compared to the last_update string. Use .get() with truthiness check to skip None timestamps in incremental tracking. --- github_backup/github_backup.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/github_backup/github_backup.py b/github_backup/github_backup.py index ada2d40..4d5394e 100644 --- a/github_backup/github_backup.py +++ b/github_backup/github_backup.py @@ -1772,9 +1772,9 @@ def backup_repositories(args, output_directory, repositories): last_update = "0000-00-00T00:00:00Z" for repository in repositories: - if "updated_at" in repository and repository["updated_at"] > last_update: + if repository.get("updated_at") and repository["updated_at"] > last_update: last_update = repository["updated_at"] - elif "pushed_at" in repository and repository["pushed_at"] > last_update: + elif repository.get("pushed_at") and repository["pushed_at"] > last_update: last_update = repository["pushed_at"] if repository.get("is_gist"):