Hi Ludo, On 29 May 2026 16:01:16 CEST, "Ludovic Courtès" <[email protected]> wrote: >Hello, > >Maxim Cournoyer <[email protected]> skribis: > >> I'd like to see if we can build consensus on the removal of the >> Change-Id git trailer, as proposed in >> <https://codeberg.org/guix/guix/pulls/1740> (and hinted at in GCD-003). > >Fine with me! > >While we’re at it, I would also like us to settle on the use of a >“Merges” trailer, as in: > > Merges: guix/guix#123 > >or: > > Merges: #123 > >This is useful info for anyone who’d like to see the conversation that >led to the merge. Perhaps eventually Forgejo will honor it to >automatically mark pull requests as closed—that would be ideal. > >Thoughts?
Agreed; however there is still this issue with PRs with multiple commits, what to do in such cases? Sometimes I find myself not knowing what PR a commit came from since it is not the last one in the series (and we have this problem of not everyone using it, so it's hard to know if going upwards is going to serve one with the correct PR) I think we should also document Fixes/Closes, encouraging commiters and contributors to use these both in commit messages and PR bodies. Rutherther > >Ludo’.
