News for hackers. Stuff that matters.

A comment on Git: Please Stop Squash Merging. Parent comment

Re:Git: Please Stop Squash Merging by Jcowell on Monday July 01, 2024 @01:38AM (#40842860)
I never keep the comments of the commit when squash merging. Only a sensible title of the PR and the number so it can be found. All the details are then in the PR where the more nuanced discussion is had.
[ Parent | on HN ]
Re:Git: Please Stop Squash Merging by strawhatguy on Tuesday July 02, 2024 @08:11PM (#40861667)
If you're using squash merging, yes. But I find it better to do merge commits anyway. I mean doctor up your PR so it's one commit and you can hide your mistakes, I mean, development process, before you open the PR, of course, but the button on your git-hosting tool should just be a 'merge'.

As an aside, I find myself doing amend commits all the time these days.

[ Parent | on HN ]