[PATCH] builtin/maintenance: accept "none" as a maintenance strategy

[PATCH] builtin/maintenance: accept "none" as a maintenance strategy

[David Lin]

Commit d465be2327 (builtin/maintenance: don't silently ignore invalid
strategy, 2025-10-24) changed scheduled maintenance to error on an
unknown maintenance strategy instead of silently defaulting to the
`none` strategy.

However, `parse_maintenance_strategy()` does not recognize `none`, so
Git rejects a valid and documented strategy that can be used to override
an existing strategy and disable maintenance tasks.

Accept `none` as a valid maintenance strategy and add tests to ensure
it's accepted.

Signed-off-by: David Lin <davidlin@stripe.com>
---
 builtin/gc.c           | 2 ++
 t/t7900-maintenance.sh | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/builtin/gc.c b/builtin/gc.c
index 46999a99ab..3d1e39d46a 100644
--- a/builtin/gc.c
+++ b/builtin/gc.c
@@ -1922,6 +1922,8 @@ static const struct maintenance_strategy geometric_strategy = {
 
 static struct maintenance_strategy parse_maintenance_strategy(const char *name)
 {
+	if (!strcasecmp(name, "none"))
+		return none_strategy;
 	if (!strcasecmp(name, "incremental"))
 		return incremental_strategy;
 	if (!strcasecmp(name, "gc"))
diff --git a/t/t7900-maintenance.sh b/t/t7900-maintenance.sh
index a8d691719d..130c971b15 100755
--- a/t/t7900-maintenance.sh
+++ b/t/t7900-maintenance.sh
@@ -1022,6 +1022,9 @@ test_expect_success 'maintenance.strategy is respected' '
 		test_must_fail git -c maintenance.strategy=unknown maintenance run 2>err &&
 		test_grep "unknown maintenance strategy: .unknown." err &&
 
+		test_strategy none </dev/null &&
+		test_strategy none --schedule=weekly </dev/null &&
+
 		test_strategy incremental <<-\EOF &&
 		git pack-refs --all --prune
 		git reflog expire --all

base-commit: 13c7afec212fc97ce257d15601659314c6673d6c
-- 
2.54.0