From: Harald Nordgren via GitGitGadget Date: Sun, 04 Oct 2026 08:34:46 GMT Subject: [PATCH v2] branch: let --delete-merged default to every upstream Message-ID: In-Reply-To: From: Harald Nordgren Cleaning up every branch whose work has landed upstream required typing '**' as the pattern. Let a bare "git branch --delete-merged" consider every upstream. This applies only when the option comes last, so "git branch --dry-run --delete-merged" previews the cleanup. Signed-off-by: Harald Nordgren --- branch: let --delete-merged default to every upstream A bare git branch --delete-merged now considers every upstream. Changes in v2: * Name the default pattern ** instead of */*. Drop reference to --merged. Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-2428%2FHaraldNordgren%2Fbranch-delete-merged-default-v2 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-2428/HaraldNordgren/branch-delete-merged-default-v2 Pull-Request: https://github.com/git/git/pull/2428 Range-diff vs v1: 1: 9a576d941c ! 1: c496777c0b branch: let --delete-merged default to every upstream @@ Commit message branch: let --delete-merged default to every upstream Cleaning up every branch whose work has landed upstream required - typing '*/*' as the pattern. + typing '**' as the pattern. - Let a bare "git branch --delete-merged" consider every upstream. As - with "--merged" without a commit, this applies only when the option - comes last, so "git branch --dry-run --delete-merged" previews the - cleanup. + Let a bare "git branch --delete-merged" consider every upstream. This + applies only when the option comes last, so + "git branch --dry-run --delete-merged" previews the cleanup. Signed-off-by: Harald Nordgren Documentation/git-branch.adoc | 6 ++++-- builtin/branch.c | 14 +++++++++++--- t/t3200-branch.sh | 25 ++++++++++++++++++++++--- 3 files changed, 37 insertions(+), 8 deletions(-) diff --git a/Documentation/git-branch.adoc b/Documentation/git-branch.adoc index bfdf459329..4a91ae6879 100644 --- a/Documentation/git-branch.adoc +++ b/Documentation/git-branch.adoc @@ -25,6 +25,7 @@ git branch (-m|-M) [] git branch (-c|-C) [] git branch (-d|-D) [-r] ... git branch --edit-description [] +git branch [--dry-run] --delete-merged git branch [--dry-run] (--delete-merged )... [...] DESCRIPTION @@ -202,14 +203,15 @@ This option is only applicable in non-verbose mode. Print the name of the current branch. In detached `HEAD` state, nothing is printed. -`--delete-merged `:: +`--delete-merged []`:: Delete local branches whose configured upstream matches __, but only when their tip is reachable from that upstream. In other words, the work on the branch has already landed on the upstream it tracks, so the local copy is no longer needed. __ may name a ref, a remote (using the branch its `HEAD` points at), or a shell-style glob. The option can be - repeated to widen the upstream match. + repeated to widen the upstream match. Without __, every + upstream matches. Optional __ arguments limit which local branches are considered, e.g. `git branch --delete-merged 'origin/*' 'topic-*'`. diff --git a/builtin/branch.c b/builtin/branch.c index a613148fc7..f2a4e117dc 100644 --- a/builtin/branch.c +++ b/builtin/branch.c @@ -39,6 +39,7 @@ static const char * const builtin_branch_usage[] = { N_("git branch [] (-c | -C) [] "), N_("git branch [] [-r | -a] [--points-at]"), N_("git branch [] [-r | -a] [--format]"), + N_("git branch [] --delete-merged"), N_("git branch [] (--delete-merged )... " "[...]"), NULL @@ -1029,9 +1030,16 @@ int cmd_branch(int argc, OPT_BOOL(0, "create-reflog", &reflog, N_("create the branch's reflog")), OPT_BOOL(0, "edit-description", &edit_description, N_("edit the description for the branch")), - OPT_CALLBACK_F(0, "delete-merged", &delete_merged, N_("pattern"), - N_("delete merged branches whose upstream matches (repeatable)"), - PARSE_OPT_NONEG, parse_opt_strvec), + { + .type = OPTION_CALLBACK, + .long_name = "delete-merged", + .value = &delete_merged, + .argh = N_("pattern"), + .help = N_("delete merged branches whose upstream matches (repeatable)"), + .flags = PARSE_OPT_LASTARG_DEFAULT | PARSE_OPT_NONEG, + .callback = parse_opt_strvec, + .defval = (intptr_t) "**", + }, OPT_BOOL(0, "dry-run", &dry_run, N_("with --delete-merged, only print which branches would be deleted")), OPT__FORCE(&force, N_("force creation, move/rename, deletion"), PARSE_OPT_NOCOMPLETE), diff --git a/t/t3200-branch.sh b/t/t3200-branch.sh index cdb6c6a634..e60f4794c8 100755 --- a/t/t3200-branch.sh +++ b/t/t3200-branch.sh @@ -2133,9 +2133,28 @@ test_expect_success '--delete-merged result is independent of stacked branch nam ) ' -test_expect_success '--delete-merged requires a value' ' - test_must_fail git -C forked branch --delete-merged 2>err && - test_grep "requires a value" err +test_expect_success '--delete-merged without a pattern matches every upstream' ' + setup_repo_for_delete_merged && + create_merged_branch merged && + ( + cd repo && + git branch --track local-topic main && + git checkout --detach && + + git branch --dry-run --delete-merged && + + check_branches <<-\EOF && + local-topic + main + merged + EOF + + git branch --delete-merged && + + check_branches <<-\EOF + main + EOF + ) ' test_expect_success '--delete-merged honours branch..deleteMerged=false' ' base-commit: c46c1e37724f0478939de636ab8ea5a89086d532 -- gitgitgadget