From 20e9e299757a42c59aac690ff741589ac993515b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Steven!=20Ragnaro=CC=88k?= Date: Wed, 25 Jun 2025 11:18:34 -0700 Subject: [PATCH] Fix no remote check. This check has likely never worked as intended. I'm not aware of any case where `len()` returns a negative nubmer. The described error seems like the intent was to check for an empty list of remotes not a somehow negative number of remotes. When updating this check, there are several test failures which i have not triaged. It's possible that now-expected behavior is incompatible with this remote check or that our test fixtures need to be updated so they don't cause this error. --- bloom/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bloom/config.py b/bloom/config.py index 94287f0e..677eed7b 100644 --- a/bloom/config.py +++ b/bloom/config.py @@ -330,7 +330,7 @@ def check_for_multiple_remotes(): if get_root() is None: return remotes = get_remotes() - if len(remotes) < 0: + if len(remotes) == 0: error("Current git repository has no remotes. " "If you are running bloom-release, please change directories.", exit=True)