Ignore force pushes for changed files in a PR review (#34837)

Fixes #34832

---------

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
This commit is contained in:
delvh 2025-06-24 17:22:32 +02:00 committed by GitHub
parent eb87e9d3b6
commit 9a23fe131c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -1356,6 +1356,7 @@ func SyncUserSpecificDiff(ctx context.Context, userID int64, pull *issues_model.
// But as that does not work for all potential errors, we simply mark all files as unchanged and drop the error which always works, even if not as good as possible
if err != nil {
log.Error("Could not get changed files between %s and %s for pull request %d in repo with path %s. Assuming no changes. Error: %w", review.CommitSHA, latestCommit, pull.Index, gitRepo.Path, err)
err = nil //nolint
}
filesChangedSinceLastDiff := make(map[string]pull_model.ViewedState)
@ -1397,7 +1398,7 @@ outer:
}
}
return review, err
return review, nil
}
// CommentAsDiff returns c.Patch as *Diff