Re:Abandoning Gitflow and GitHub in favour of Gerrit
by HolyHaddock on Tuesday April 05, 2016 @12:34PM
(#11431852)
> Yes, code review is an invaluable part of the git history. It explains why you made the decisions you made.
This is interesting - even places I've worked that have cared about commit messages have been pretty happy with the entirely content-free "Fixes from code review."
Do you summarise code review discussions in the commit messages?
Re:Abandoning Gitflow and GitHub in favour of Gerrit
by Touche on Tuesday April 05, 2016 @12:54PM
(#11432062)
Yes, I do, I try to treat each commit like something that can be read in isolation. But nevertheless, the merge commit will have (if using GitHub) a reference to the pull request where you can see the discussion, which is valuable imo.
Re:Abandoning Gitflow and GitHub in favour of Gerrit
by HolyHaddock on Thursday April 07, 2016 @05:28AM
(#11445942)
Have you had any feedback from colleagues that have found that useful?
I'm not following this at all... what is it you need from this tool and why?
> Does it matter that you had to implement 12 fixups at various points and rewrite the whole feature thrice after requirements changed? Probably not.
Yes, code review is an invaluable part of the git history. It explains why you made the decisions you made. And having many small commits rather than 1 massive one makes it much easier to revert bad commits.