Skip to content

Commit f88715e

Browse files
bhalbrightlafrikslunny
authored andcommitted
Trigger webhook when deleting a branch after merging a PR (#9424)
* fixed so when deleting a branch after merging a PR, a webhook is triggered * changed to use pr.HeadRepo instead of ctx.Repo when sending webhook for a deletion of branch after merging pr Co-authored-by: Lauris BH <[email protected]> Co-authored-by: Lunny Xiao <[email protected]>
1 parent 8e65572 commit f88715e

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

routers/repo/pull.go

+16
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import (
2222
"code.gitea.io/gitea/modules/git"
2323
"code.gitea.io/gitea/modules/log"
2424
"code.gitea.io/gitea/modules/notification"
25+
"code.gitea.io/gitea/modules/repofiles"
2526
"code.gitea.io/gitea/modules/setting"
2627
"code.gitea.io/gitea/modules/util"
2728
"code.gitea.io/gitea/services/gitdiff"
@@ -928,6 +929,21 @@ func CleanUpPullRequest(ctx *context.Context) {
928929
return
929930
}
930931

932+
if err := repofiles.PushUpdate(
933+
pr.HeadRepo,
934+
pr.HeadBranch,
935+
repofiles.PushUpdateOptions{
936+
RefFullName: git.BranchPrefix + pr.HeadBranch,
937+
OldCommitID: branchCommitID,
938+
NewCommitID: git.EmptySHA,
939+
PusherID: ctx.User.ID,
940+
PusherName: ctx.User.Name,
941+
RepoUserName: pr.HeadRepo.Owner.Name,
942+
RepoName: pr.HeadRepo.Name,
943+
}); err != nil {
944+
log.Error("Update: %v", err)
945+
}
946+
931947
if err := models.AddDeletePRBranchComment(ctx.User, pr.BaseRepo, issue.ID, pr.HeadBranch); err != nil {
932948
// Do not fail here as branch has already been deleted
933949
log.Error("DeleteBranch: %v", err)

0 commit comments

Comments
 (0)