This document is lifted straight from Google's engineering practices with some tweaks for things that make sense to use here at Latitude.

Currently this contains the following documents:

• Latitude's Code Review Guidelines, which are actually two separate sets of documents:
  • The Code Reviewer's Guide
  • The Change Author's Guide

Term Clarification:

• CL: Stands for "changelist", which means one self-contained change that has been submitted to version control or which is undergoing code review. Other organizations often call this a "change", "patch", or "pull-request".
• LGTM: Means "Looks Good to Me". It is what a code reviewer says when approving a CL.
• NIT: Means "nitpick" which is where a reviewer is making a comment on a CL but it is only meant as a mild suggestion rather than a request for change.

The documents in this project are licensed under the CC-By 3.0 License, which encourages you to share these documents. See https://creativecommons.org/licenses/by/3.0/ for more details.