This should be much more wide-spread. The hardest part of programming is reading someone else’s code.
More people should learn to do git rebase -i
, it’s a simple way to re-organise your commits to make sure that they tell a story to someone going through the PR commit by commit. It only takes a minute and can save your colleagues so much time and increase the quality of the review process.
Still, for those very hard bugs that everyone just hovers around and procrastinates, this is perfect advice!