Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

delete a merged pull request return 500 #19655

Closed
lunny opened this issue May 8, 2022 · 3 comments · Fixed by #20032
Closed

delete a merged pull request return 500 #19655

lunny opened this issue May 8, 2022 · 3 comments · Fixed by #20032
Labels

Comments

@lunny
Copy link
Member

lunny commented May 8, 2022

exit status 1 - error: cannot lock ref 'refs/pull/2': 'refs/pull/2/head' exists; cannot create 'refs/pull/2'
 - error: cannot lock ref 'refs/pull/2': 'refs/pull/2/head' exists; cannot create 'refs/pull/2'
@lunny lunny added the type/bug label May 8, 2022
@a1012112796
Copy link
Member

we can delete pull request now?

@lunny
Copy link
Member Author

lunny commented May 9, 2022

Yes.

@a1012112796
Copy link
Member

I see. ref: #18879

a1012112796 added a commit to a1012112796/gitea that referenced this issue Jun 19, 2022
fix go-gitea#19655

Signed-off-by: a1012112796 <1012112796@qq.com>
lunny pushed a commit that referenced this issue Jun 19, 2022
* fix delete pull head ref for DeleteIssue

fix #19655

Signed-off-by: a1012112796 <1012112796@qq.com>

* add different help message for delete pull request

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: 6543 <6543@obermui.de>
a1012112796 added a commit to a1012112796/gitea that referenced this issue Jun 19, 2022
* fix delete pull head ref for DeleteIssue

fix go-gitea#19655

Signed-off-by: a1012112796 <1012112796@qq.com>

* add different help message for delete pull request

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: 6543 <6543@obermui.de>
zeripath pushed a commit that referenced this issue Jun 19, 2022
Backport #20032 

In DeleteIssue the PR git head reference should be `/refs/pull/xxx/head` not `/refs/pull/xxx`

Fix #19655

Signed-off-by: a1012112796 <1012112796@qq.com>
vsysoev pushed a commit to IntegraSDL/gitea that referenced this issue Aug 10, 2022
* fix delete pull head ref for DeleteIssue

fix go-gitea#19655

Signed-off-by: a1012112796 <1012112796@qq.com>

* add different help message for delete pull request

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: 6543 <6543@obermui.de>
AbdulrhmnGhanem pushed a commit to kitspace/gitea that referenced this issue Aug 24, 2022
* fix delete pull head ref for DeleteIssue

fix go-gitea#19655

Signed-off-by: a1012112796 <1012112796@qq.com>

* add different help message for delete pull request

Signed-off-by: a1012112796 <1012112796@qq.com>

Co-authored-by: 6543 <6543@obermui.de>
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants