fix: branch checking logic

See https://github.com/go-git/go-git/issues/518 for why this is needed.
This commit is contained in:
Franz Heinzmann (Frando) 2022-08-11 14:14:31 +02:00 committed by decentral1se
parent b42d5bf113
commit 8458e61d17
Signed by: decentral1se
GPG Key ID: 03789458B3D0C410
2 changed files with 21 additions and 2 deletions

View File

@ -5,6 +5,25 @@ import (
"github.com/go-git/go-git/v5/plumbing"
)
// Check if a branch exists in a repo.
// Use this and not repository.Branch(), because the latter does not
// actually check for existing branches.
// See https://github.com/go-git/go-git/issues/518
func HasBranch(repository *git.Repository, name string) bool {
var exist bool
if iter, err := repository.Branches(); err == nil {
iterFunc := func(reference *plumbing.Reference) error {
if name == reference.Name().Short() {
exist = true
return nil
}
return nil
}
_ = iter.ForEach(iterFunc)
}
return exist
}
// GetCurrentBranch retrieves the current branch of a repository
func GetCurrentBranch(repository *git.Repository) (string, error) {
branchRefs, err := repository.Branches()

View File

@ -625,8 +625,8 @@ func GetDefaultBranch(repo *git.Repository, recipeName string) (plumbing.Referen
}
branch := "master"
if _, err := repo.Branch("master"); err != nil {
if _, err := repo.Branch("main"); err != nil {
if !gitPkg.HasBranch(repo, "master") {
if !gitPkg.HasBranch(repo, "main") {
return "", fmt.Errorf("failed to select default branch in %s", recipeDir)
}
branch = "main"