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

Backport #34837 by delvh

Fixes #34832

Co-authored-by: delvh <dev.lh@web.de>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
This commit is contained in:
authored and GitHub committed 2025-06-24 16:41:07 +00:00
1 parent 0bfccd8ecf
commit 1e69f085d6
1 file changed
+2 -1
+2 -1
View File
@@ -1359,6 +1359,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)
@@ -1400,7 +1401,7 @@ outer:
}
}
return review, err
return review, nil
}
// CommentAsDiff returns c.Patch as *Diff