Re:Code Reviews and Bad Habits
by aiiane on Thursday February 20, 2014 @12:18PM
(#7272074)
Side by side diffs, automatic integration, transactional commenting (queue up a bunch of comments, then publish them with a single notification to the patch writer), enforcement of LGTMs, better visualization of the evolution of a patch, better tracking of resolved comments, etc.
[ Parent
| on HN ]
Thanks for the info, I'm glad someone responded. Bitbucket has side by side diffs, but I think both those and inline difs work fine (just a matter of taste).
I don't think those other features (for me) justify the complexity of running another application. For example, not enforcing LGTMs is a feature, in am emergency, a team member can get stuff done. But if someone abuses that regularly, they loose pull rights (or are fired).