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

Merge conflicts block the 'Mark as Manually Merged' operation #24729

Closed
drsybren opened this issue May 15, 2023 · 5 comments
Closed

Merge conflicts block the 'Mark as Manually Merged' operation #24729

drsybren opened this issue May 15, 2023 · 5 comments
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail type/bug

Comments

@drsybren
Copy link
Contributor

Description

When a PR shows this message at the bottom:

This pull request has changes conflicting with the target branch.

it is no longer possible to mark the PR as 'manually merged'. This is the same for when the conflict checking is in progress, in such cases it's also impossible to just mark things as merged. Since conflict checking can take a lot of time, this is often a blocking issue for the Blender project.

Gitea Version

https://github.com/blender/gitea/tree/blender-merged tracking Gitea main at ce5d711

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

Running our own branch.

Database

PostgreSQL

@silverwind
Copy link
Member

silverwind commented May 15, 2023

it is no longer possible to mark the PR as 'manually merged'.

You think it's an regression? We did alter the PR merge box recently, so I suspect it came from that work, specifically #23951.

@drsybren
Copy link
Contributor Author

I'm not sure whether it's a regression or not. In any case I feel that it would be prudent to always have the option to close a PR as 'manually merged', regardless of any background processing that might be going on.

@wxiaoguang
Copy link
Contributor

I guess it is a regression, I have left comments there

image

@wxiaoguang wxiaoguang modified the milestone: 1.20.0 Jul 9, 2023
@wxiaoguang
Copy link
Contributor

On my side, I can see the "manual merge" form.

Do you have a reproducible example?

image

@wxiaoguang wxiaoguang added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Jul 9, 2023
@drsybren
Copy link
Contributor Author

I think this already got resolved in the mean time.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail type/bug
Projects
None yet
Development

No branches or pull requests

3 participants