Skip to content

Conversation

GiteaBot
Copy link
Collaborator

Backport #23683 by @sillyguodong

Fix: #23674
If the type of hook_event is pull_request_sync, also need to insert a record of commit_status into DB.
Because pull_request event and pull_request_sync event have the same payload, so the code is reusable.

Screenshot:
image

Fix: go-gitea#23674
If the type of `hook_event` is `pull_request_sync`, also need to insert
a record of `commit_status` into DB.
Because `pull_request` event and `pull_request_sync` event have the same
payload, so the code is reusable.

Screenshot:

![image](https://user-images.githubusercontent.com/33891828/227465436-1b436807-d0b2-4a2b-8890-09d96c7f666d.png)
@GiteaBot GiteaBot added this to the 1.19.1 milestone Mar 24, 2023
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 24, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 24, 2023
@jolheiser jolheiser added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 24, 2023
@techknowlogick techknowlogick enabled auto-merge (squash) March 24, 2023 20:32
@jolheiser
Copy link
Member

🎺 🤖

@techknowlogick techknowlogick merged commit a9b4aa4 into go-gitea:release/v1.19 Mar 24, 2023
@lunny lunny removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 25, 2023
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
@techknowlogick techknowlogick deleted the backport-23683-v1.19 branch September 9, 2023 05:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants