Skip to content

[Feature] Auto-lock PR Conversations When Merged #22121

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

Open
kdumontnu opened this issue Dec 13, 2022 · 6 comments
Open

[Feature] Auto-lock PR Conversations When Merged #22121

kdumontnu opened this issue Dec 13, 2022 · 6 comments
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active. type/proposal The new feature has not been accepted yet but needs to be discussed first.

Comments

@kdumontnu
Copy link
Contributor

Feature Description

I think it would be helpful if repos had a setting to automatically lock a conversation when an issue is closed and/or PR is merged.

Related to: #20012

Screenshots

No response

@kdumontnu kdumontnu added type/proposal The new feature has not been accepted yet but needs to be discussed first. type/feature Completely new functionality. Can only be merged if feature freeze is not active. labels Dec 13, 2022
@delvh
Copy link
Member

delvh commented Dec 13, 2022

Wouldn't it be better to let a CI do that?
Then we don't have a weird edge case that is only sometimes wanted in our core while still providing the functionality…
As far as I know, the CI currently proposed should (later on) be able to do that (see #21937)

@kdumontnu
Copy link
Contributor Author

Wouldn't it be better to let a CI do that? Then we don't have a weird edge case that is only sometimes wanted in our core while still providing the functionality… As far as I know, the CI currently proposed should (later on) be able to do that (see #21937)

Yeah, I appreciate your point, but CI seems like overkill for this if a user isn't already running CI for something else (also, is there a webhook for issues closed or would the CI job it have to poll each open issue?). It seems like what you suggest is the strategy GitHub has taken, as there doesn't seem to be a feature like this built in. From a UX perspective, it seems like like a nice compliment to auto-merge.

If this really is a niche feature that a majority of users don't want/need, I agree it should go to CI. If a reasonable number of folks want this it would be nice to build into the UI. I'll post in discord and see if other people would like this feature, and if there's not much traction in a week then I'm perfectly happy closing this.

@jolheiser
Copy link
Member

jolheiser commented Dec 14, 2022

also, is there a webhook for issues closed or would the CI job it have to poll each open issue?

Yes-ish, the base issues covers "Issue opened, closed, reopened, or edited."
The job would need to check if it was closed specifically, but the event would come across.

@lunny
Copy link
Member

lunny commented Dec 14, 2022

Wouldn't it be better to let a CI do that? Then we don't have a weird edge case that is only sometimes wanted in our core while still providing the functionality… As far as I know, the CI currently proposed should (later on) be able to do that (see #21937)

Yeah, I appreciate your point, but CI seems like overkill for this if a user isn't already running CI for something else (also, is there a webhook for issues closed or would the CI job it have to poll each open issue?). It seems like what you suggest is the strategy GitHub has taken, as there doesn't seem to be a feature like this built in. From a UX perspective, it seems like like a nice compliment to auto-merge.

If this really is a niche feature that a majority of users don't want/need, I agree it should go to CI. If a reasonable number of folks want this it would be nice to build into the UI. I'll post in discord and see if other people would like this feature, and if there's not much traction in a week then I'm perfectly happy closing this.

That Action could be reused when Gitea Actions PR merged.

@a1012112796
Copy link
Member

a1012112796 commented Dec 15, 2022

How about add an extern option to do it ?
屏幕截图 2022-12-14 170346

the quick_actions in gitlab looks also good.

@lunny
Copy link
Member

lunny commented May 7, 2024

Not a good idea to put more and more checkboxes/buttons there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature Completely new functionality. Can only be merged if feature freeze is not active. type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

No branches or pull requests

5 participants