From 7f2733fa1b412593f676cd04a1c9166b51e56640 Mon Sep 17 00:00:00 2001 From: Josh Frye Date: Thu, 3 Mar 2016 12:43:23 -0500 Subject: [PATCH] Return errors instead of just logging them. --- models/repo.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/models/repo.go b/models/repo.go index 30abc5d430..22336d3c1a 100644 --- a/models/repo.go +++ b/models/repo.go @@ -676,13 +676,11 @@ func MigrateRepository(u *User, opts MigrateRepoOptions) (*Repository, error) { // Try to get HEAD branch and set it as default branch. gitRepo, err := git.OpenRepository(repoPath) if err != nil { - log.Error(4, "OpenRepository: %v", err) - return repo, nil + return repo, fmt.Errorf("OpenRepository: %v", err) } headBranch, err := gitRepo.GetHEADBranch() if err != nil { - log.Error(4, "GetHEADBranch: %v", err) - return repo, nil + return repo, fmt.Errorf("GetHEADBranch: %v", err) } if headBranch != nil { repo.DefaultBranch = headBranch.Name