Disclaimer: I’m not representing Amazon in any way. Opinions written here are strictly my own.

I’ve code reviewed over 750 pull requests in my 5-year software engineer career at Amazon. As my team’s tech lead I provide insightful feedback and encourage a high code quality bar.

But as a junior engineer I was a poor reviewer. I didn’t know where to start, what to look for or how to comment. I gave useless comments on style. I rushed through reviews giving the classic “LGTM!” after missing obvious flaws.

What follows is my thought process on how I review pull requests…

Curtis Einsmann

Lead software engineer at Amazon Web Services. Opinions mine. All stories free. Follow me on Twitter for more: https://twitter.com/curtiseinsmann

Get the Medium app

A button that says 'Download on the App Store', and if clicked it will lead you to the iOS App store
A button that says 'Get it on, Google Play', and if clicked it will lead you to the Google Play store