forks/forgejo
1
0
Fork 0
Code Releases Activity

Merge pull request #2087 from Bwko/fix_error_exit

Browse source 
Fix exit status 1 not handled @ getMergeCommit (#2087)
This commit is contained in:
Lauris BH 2017-06-30 15:46:49 +03:00 committed by GitHub
commit 12cb6cd3c9
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
models/pull.go
View file

@ -495,7 +495,7 @@ func (pr *PullRequest) getMergeCommit() (*git.Commit, error) {
if err != nil { if err != nil {
// Errors are signaled by a non-zero status that is not 1 // Errors are signaled by a non-zero status that is not 1
if err.Error() == "exit status 1" { if strings.Contains(err.Error(), "exit status 1") {
return nil, nil return nil, nil
} }
return nil, fmt.Errorf("git merge-base --is-ancestor: %v %v", stderr, err) return nil, fmt.Errorf("git merge-base --is-ancestor: %v %v", stderr, err)