Surviving Code ReviewsLorna Mitchell
ROME 18-19 MARCH 2016
Surviving Code ReviewsLorna Mitchell, CodeMotion Rome 2016
If there aren't anyacceptance criteria for
your patch, invent some
Committing commentedcode is like leaving litter
on a campsite
Your commit history willbe part of my projectuntil the end of time
git reset --soft $(git merge-base master HEAD)
http://chris.beams.io/posts/git-commit/
git merge --no-commit --no-ff feature
git merge --abort
A pull request is the startof a conversation about a
feature
peer review vs gatekeepers
Being able to see whatyou're NOT looking at isa reviewer's superpower
colleagues vs contributors
Pull requests have alifecycle, be prepared tochampion them to the
end
Thankyouhttp://lornajane.net
This work is licensed under a Creative Commons
Attribution-NonCommercial-ShareAlike 3.0Unported License.