Effective Code Review

Michael  Mike Sherov
Head Director of Engineering
Behance Team @ Adobe

Reviewers: code review is feedback, deliver it as such

Reviewers: focus on the goals, don't nitpick

Reviewers: offer substantive suggestions

Reviewers: it's ok to
say "pass"

Reviewers: bias towards approval

Authors: Code Review is feedback, receive it as such

Authors: Get feedback early, before you code

Authors: keep it small and readable

Authors: refactor early and often, don't wait

What questions do you have?

Effective Code Review

By mikesherov

Effective Code Review

  • 1,328