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