Project: mesos
Issue MESOS-2581: Document tips, best practices, guidelines for doing code reviews.
We currently have a ["Committers Guide"|https://github.com/apache/mesos/blob/0.22.0/docs/committers-guide.md], however most of this information is relevant to all contributors looking to be participating in the code review process. I'm proposing we extract much of this information into a more general "Code Reviewing" document, and include additional tips, best practices, lessons learned from members of the community. This would be a great pre-requisite for on-boarding more committers and adding [MAINTAINERS|http://mail-archives.apache.org/mod_mbox/mesos-dev/201502.mbox/%3CCA+8RcoReugMVqoOpsnB8WGYBELa5fHwPA=J=YHJE22iwZvsbeQ@mail.gmail.com%3E]. The committers guide can be more specific to our expectations of committers, so we may want to make this into a "committership" document to help set expectations for contributors looking to become committers.
Story points: 3