`git delete-merged-branches` can delete default branch such as `master`

[bric3]

`git delete-merged-branches` can delete default branch such as `master`

[spacewander]

It seems that it is because the default branch is main? There is a check in the cmd:

git-extras/bin/git-delete-merged-branches

Line 3 in 5f19424

branches=$(git branch --no-color --merged | grep -vE "^(\*|\+)" | grep -v "$(git_extra_default_branch)" | grep -v svn)

Either you can configure the default branch to master, or the default branch detection is not correct.

[hyperupcall]

@spacewander I think it would be simple and effective to skip the deletion of branches with hardcoded values of master, main, trunk, default, development, stable, release, and production. I think that should detect 99% of cases, and will ignore some useful branches that may not be the default branch of any remote. To prevent confusion over the change of behavior, I think a warning should be emitted if we ignore a branch.

We can also read the config value for the default init branch and add that.

Indeed, I suggest the following approach

If there's a single remote use it
If multiple, only look up origin, then maybe upstream.
If not possible look at the existing init default branch

I agree with @spacewander, although it is a good heuristic, I think there is too much uncertainty around it and it will invariably do different things, even on the same computer. I for one, always use me instead of origin.

Another idea is to loop over all remotes, and add any of their default branches to the ignore list. I am not completely certain that it will not introduce many false positives.

[bric3]

It seems the git_extra_default_branch help function work by looking at configured values or environment variables, if none are present this function falls back to main.

git-extras/helper/git-extra-utility

Lines 8 to 19 in 5f19424

	git_extra_default_branch() {
	local extras_default_branch init_default_branch
	extras_default_branch=$(git config --get git-extras.default-branch)
	init_default_branch=$(git config --get init.defaultBranch)
	if [ -n "$extras_default_branch" ]; then
	echo "$extras_default_branch"
	elif [ -n "$init_default_branch" ]; then
	echo "$init_default_branch"
	else
	echo "main"
	fi
	}

[spacewander]

So you can configure the default branch to master, if you are using it as the default branch.

[bric3]

Yes but wouldn't that be better if git-extras discover this automatically, using the commands above ?

[spacewander]

Yes but wouldn't that be better if git-extras discover this automatically, using the commands above ?

Would you happen to have any suggestions? The git_extra_default_branch is just reading the configured value.

[bric3]

@spacewander Sorry for the late reply. Yes I think what I proposed in the top message could help resolve the default branch :

git rev-parse --abbrev-ref ${the_remote}/HEAD

[spacewander]

@spacewander Sorry for the late reply. Yes I think what I proposed in the top message could help resolve the default branch :

git rev-parse --abbrev-ref ${the_remote}/HEAD

Err... But there may be multiple remotes.

[bric3]

Indeed, I suggest the following approach

1. If there's a single remote use it
2. If multiple, only look up origin, then maybe upstream.
3. If not possible look at the existing init default branch

[spacewander]

Indeed, I suggest the following approach

1. If there's a single remote use it
2. If multiple, only look up origin, then maybe upstream.
3. If not possible look at the existing init default branch

It seems too complex to me, which may introduce strange behavior. For example, the git-extras in my local machine has two remotes:

origin	[email protected]:spacewander/git-extras (fetch)
origin	[email protected]:spacewander/git-extras (push)
own	[email protected]:tj/git-extras (fetch)
own	[email protected]:tj/git-extras (push)

The origin(https://github.com/spacewander/git-extras) 's default branch is master, while the own(https://github.com/tj/git-extras) 's default branch is main.

The guess can't guarantee it's 100% correct.

@hyperupcall @vanpipy
What about your opinion?