Skip to content

Commit

Permalink
pull: ignore PR status check if head repository is missing (#7004)
Browse files Browse the repository at this point in the history
  • Loading branch information
unknwon committed Jun 4, 2022
1 parent 75e1044 commit 6982749
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions internal/db/pull.go
Expand Up @@ -621,11 +621,6 @@ func (pr *PullRequest) UpdateCols(cols ...string) error {

// UpdatePatch generates and saves a new patch.
func (pr *PullRequest) UpdatePatch() (err error) {
if pr.HeadRepo == nil {
log.Trace("PullRequest[%d].UpdatePatch: ignored corrupted data", pr.ID)
return nil
}

headGitRepo, err := git.Open(pr.HeadRepo.RepoPath())
if err != nil {
return fmt.Errorf("open repository: %v", err)
Expand Down Expand Up @@ -759,6 +754,11 @@ func (prs PullRequestList) LoadAttributes() error {

func addHeadRepoTasks(prs []*PullRequest) {
for _, pr := range prs {
if pr.HeadRepo == nil {
log.Trace("addHeadRepoTasks[%d]: missing head repository", pr.ID)
continue
}

log.Trace("addHeadRepoTasks[%d]: composing new test task", pr.ID)
if err := pr.UpdatePatch(); err != nil {
log.Error("UpdatePatch: %v", err)
Expand Down

0 comments on commit 6982749

Please sign in to comment.