Code Reviews

💡 This template documents how to review code. Helpful for new and remote employees to get and stay aligned.

Philosophy

Why do you perform code reviews? What are your guiding principles for these reviews?

You may want to mention other pages here. Like Engineering Guidelines. To link to another page inline, type @ followed by the name of the page:

Engineering Guidelines⁠

⁠

Preparation sets your reviewers up for success.

Make sure your commit messages are descriptive.

Your PR descriptions should be an extension of your commit messages. Write about both what the commit changes, and how you implemented the change.

Make two passes over the PR if it's substantial.

On the first pass, come to an understanding of the code change at a high level.

On the second pass, pay more attention to semantic details.

var commentCount = 0;

You might suggest that this be a let instead of var.

Want to print your doc?
This is not the way.

Try clicking the ⋯ next to your doc name or using a keyboard shortcut (

CtrlP

) instead.