Correct Git commits with git-autofixup

Re:Correct Git commits with git-autofixup by zimmski on Thursday March 18, 2021 @07:04AM (#26501433)

git-autofixup definitely changed my daily development life.... I mostly use it with this Git alias

``` autofixup = !"autofixup origin/master --exit-code; test $? -lt 2 && GIT_SEQUENCE_EDITOR=true git rebase -i --autosquash $(git merge-base HEAD origin/master)" ```

Works almost every time perfectly, and when not i always get a comment during a code review.

[ Parent | on HN ]

Re:Correct Git commits with git-autofixup by jasonpeacock on Thursday March 18, 2021 @09:15AM (#26502298)

> when not i always get a comment during a code review.

Why during CR? Is the error not obvious to you? You should be able to check the results of your changes before pushing, right?

[ Parent | on HN ]

Re:Correct Git commits with git-autofixup by zimmski on Thursday March 18, 2021 @09:24AM (#26502396)

I/we do but sometimes you just overlook something especially if there are a lot of changes. But having someone else review what you do almost completely removes that problem.

I usually do not look through all commits again when a basic review already happend and i "just" integrate the review comments. That is where for me personally these problems happen that git-autofixup is sometimes wrong but in over a few thousand commits in the last months this happend only thrice. Which for me just speaks for the tool IMHO. Ow the original author big time :-)

[ Parent | on HN ]

Re:Correct Git commits with git-autofixup by torbiak on Sunday March 28, 2021 @04:31AM (#26608815)

Cool, I'm glad it's working out for you. Due to my workflow at my current job I haven't actually had much occasion to use it since writing it, so I'm not that confident about how good the hueristics are.

Do you generally use the default --strict setting?

[ Parent | on HN ]