Zulip Chat Archive

Stream: mathlib4

Topic: github question


Kevin Buzzard (May 19 2024 at 13:42):

I'm going through my github notifications and I notice that in the majority of them, two commits which I can't find (always by the same people) apparently "reference the PR" (despite the PRs being on wildly different topics). What's going on here?

e.g.

Screenshot-from-2024-05-19-14-38-27.png
Screenshot-from-2024-05-19-14-37-56.png

Yaël Dillies (May 19 2024 at 13:46):

They added the commit actually referencing the PR to their PR, probably by merging?

Rida Hamadani (May 19 2024 at 15:25):

You do it too:
pushedcommit.png

Mario Carneiro (May 19 2024 at 15:31):

This sounds kind of like how I get regular pings like https://github.com/leanprover-community/mathlib4/commit/669ad32efca36c9727c06f7c721856e86bd58006 because @Eric Wieser mentioned me in a commit which is now part of the master mainline and people periodically rebase this commit onto feature branches

Mario Carneiro (May 19 2024 at 15:32):

I'm pretty sure every occurrence of this is a mistake, and the resulting branches end up having hundreds of commits

Ruben Van de Velde (May 19 2024 at 15:33):

Suggestion: don't mention people by user name in commit messages :innocent:


Last updated: May 02 2025 at 03:31 UTC