Zulip Chat Archive
Stream: mathlib4
Topic: github "update branch" button
Ruben Van de Velde (Jun 12 2024 at 15:03):
Github has a button that allows merging master into your branch from the web ui: https://docs.github.com/assets/cb-12049/mw-1440/images/help/pull_requests/pull-request-update-branch-with-dropdown.webp
Apparently this can be turned on by maintainers: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-suggestions-to-update-pull-request-branches
How do people feel about doing that?
Eric Wieser (Jun 12 2024 at 15:26):
My experience with this button in the past is that newcomers just overclick it, because the button is presented as something that needs fixing
Eric Wieser (Jun 12 2024 at 15:26):
And so it will just increase our CI load for little benefit
Yaël Dillies (Jun 12 2024 at 15:36):
I personally think that manual merging is one step which is very efficient at teaching people git
Jon Eugster (Jun 13 2024 at 08:01):
Could such a button be just on PRs with a merge-conflict
label, or would it basically always be there (since master
is moving fast)?
If it provided a very simple way to update a branch with a trivial merge conflict, that could be handy. If it's just permanently there, Eric's point might be outweighting that.
Ruben Van de Velde (Jun 13 2024 at 08:04):
Permanently is the only option as far as I'm aware
Eric Wieser (Jun 14 2024 at 08:08):
When there is a merge conflict, GitHub already offers such a button by default
Ruben Van de Velde (Jun 14 2024 at 08:09):
Oh huh, I must be blind
Eric Wieser (Jun 14 2024 at 08:53):
If the conflicts are messy, GitHub hides the button
Last updated: May 02 2025 at 03:31 UTC