Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Automerge supports deleting branch automatically after merging#32343

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@lunny
Copy link
Member

@lunnylunny commentedOct 26, 2024
edited
Loading

Resolve#32341
Depends on#27151

  • It will display a checkbox of deleting the head branch on the pull request view page when starting an auto-merge task.
  • Add permission check before deleting the branch
  • Add delete branch comment for those closing pull requests because of head branch or base branch was deleted.
  • MergeRetargetChildrenOnMerge andAddDeletePRBranchComment intoservice.DeleteBranch.

@lunnylunny added the type/enhancementAn improvement of existing functionality labelOct 26, 2024
@lunnylunny added this to the1.23.0 milestoneOct 26, 2024
@GiteaBotGiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelOct 26, 2024
@github-actionsgithub-actionsbot added modifies/apiThis PR adds API routes or modifies them modifies/goPull requests that update Go code modifies/migrations modifies/frontend labelsOct 26, 2024
@lunnylunny marked this pull request as draftNovember 10, 2024 02:18
@GiteaBotGiteaBot added lgtm/need 1This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelsDec 25, 2024
@lunny
Copy link
MemberAuthor

last call @go-gitea/technical-oversight-committee

@GiteaBotGiteaBot added lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1This PR needs approval from one additional maintainer to be merged. labelsJan 9, 2025
@lunnylunny added the reviewed/wait-mergeThis pull request is part of the merge queue. It will be merged soon. labelJan 9, 2025
@lunnylunny merged commit39d51e7 intogo-gitea:mainJan 9, 2025
26 checks passed
@lunnylunny deleted the lunny/automerge_support_delete_branch branchJanuary 9, 2025 19:51
@GiteaBotGiteaBot removed the reviewed/wait-mergeThis pull request is part of the merge queue. It will be merged soon. labelJan 9, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull requestJan 10, 2025
* giteaofficial/main:  Fix raw file API ref handling (go-gitea#33172)  [skip ci] Updated translations via Crowdin  Fix ACME panic (go-gitea#33178)  Automerge supports deleting branch automatically after merging (go-gitea#32343)  Fix branch dropdown not display ref name (go-gitea#33159)  Add .run to gitignore (go-gitea#33175)  Fix assignee list overlapping in Issue sidebar (go-gitea#33176)  Fix pam auth test regression (go-gitea#33169)  Move repo size to sidebar (go-gitea#33155)  Fix fuzz test (go-gitea#33156)  Refactor older tests to use testify (go-gitea#33140)  [skip ci] Updated translations via Crowdin  Fix typo in gitea downloader test and add missing codebase in `ToGitServiceType` (go-gitea#33146)  Fix form width (go-gitea#33151)  Convert github.com/xanzy/go-gitlab into gitlab.com/gitlab-org/api/client-go (go-gitea#33126)  Refactor HTMLFormat, update chroma render, fix js error (go-gitea#33136)  Add missed transaction on setmerged (go-gitea#33079)
@go-giteago-gitea locked asresolvedand limited conversation to collaboratorsApr 10, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@wolfogrewolfogrewolfogre approved these changes

@wxiaoguangwxiaoguangAwaiting requested review from wxiaoguang

@yp05327yp05327Awaiting requested review from yp05327

+1 more reviewer

@Zettat123Zettat123Zettat123 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore.modifies/apiThis PR adds API routes or modifies themmodifies/frontendmodifies/goPull requests that update Go codemodifies/migrationstype/enhancementAn improvement of existing functionality

Projects

None yet

Milestone

1.24.0

Development

Successfully merging this pull request may close these issues.

Automerge doesn't respect branch deletion rule

6 participants

@lunny@wxiaoguang@wolfogre@Zettat123@yp05327@GiteaBot

[8]ページ先頭

©2009-2025 Movatter.jp