Portrait of David Findley

Code Review Strategies


Reviewing merge requests is hard, especially when you don't have a plan or good habits for reviewing code. I tend to approach most PRs at work in a haphazard way. At a given time I may be more likely to focus in on one aspect or another.

My goal with these notes is to document some of the different "angles" or "lenses" for reviewing code.