Browse Source

Return errors instead of just logging them.

Josh Frye 9 years ago
parent
commit
7f2733fa1b
1 changed files with 2 additions and 4 deletions
  1. 2 4
      models/repo.go

+ 2 - 4
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