From 35cb786ca1dd6d12963ac7067945065b271f36ad Mon Sep 17 00:00:00 2001
From: sillyguodong <33891828+sillyguodong@users.noreply.github.com>
Date: Thu, 30 Mar 2023 16:26:02 +0800
Subject: [PATCH] [Patch] Fix closed PR also triggers Webhooks and actions
 (#23782)

Fix #23707
Cause by #23189
This PR is a quick fix that, when pushing commits to closed PR, webhook
and actions also be triggered.
---
 services/pull/pull.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/services/pull/pull.go b/services/pull/pull.go
index e1d5a6f86dd4..fe2f002010b1 100644
--- a/services/pull/pull.go
+++ b/services/pull/pull.go
@@ -294,6 +294,10 @@ func AddTestPullRequestTask(doer *user_model.User, repoID int64, branch string,
 			}
 			if err == nil {
 				for _, pr := range prs {
+					if pr.Issue.IsClosed {
+						// The closed PR never trigger action or webhook
+						continue
+					}
 					if newCommitID != "" && newCommitID != git.EmptySHA {
 						changed, err := checkIfPRContentChanged(ctx, pr, oldCommitID, newCommitID)
 						if err != nil {