From d1cebb0e884a88941e27e2280e117cc04d2665fe Mon Sep 17 00:00:00 2001
From: Gusted <postmaster@gusted.xyz>
Date: Thu, 14 Mar 2024 16:15:56 +0100
Subject: [PATCH] [BUG] Use correct head commit for CODEOWNER
Backport of #2658
Regression of #2507, which switched the HEAD from `pr.GetGitRefName()`
to `pr.HeadCommitID` but it had to be `prInfo.HeadCommitID`. Resolves #2656
I was able to reproduce this locally with _some_ pull requests, haven't
been able to get a reproducer trough integration testing.
(cherry picked from commit a4cc37b46a937a1e558f9faab565a7cf12638f67)
---
models/issues/pull.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/models/issues/pull.go b/models/issues/pull.go
index f60b1c5e7a..c450b66a51 100644
--- a/models/issues/pull.go
+++ b/models/issues/pull.go
@@ -932,7 +932,7 @@ func PullRequestCodeOwnersReview(ctx context.Context, pull *Issue, pr *PullReque
}
// Use the merge base as the base instead of the main branch to avoid problems
// if the pull request is out of date with the base branch.
- changedFiles, err := repo.GetFilesChangedBetween(prInfo.MergeBase, pr.HeadCommitID)
+ changedFiles, err := repo.GetFilesChangedBetween(prInfo.MergeBase, prInfo.HeadCommitID)
if err != nil {
return err
}