Differential D89 Diff 207 source/contributor-guide/peer-review.rst

Changeset View

Standalone View

				.. _contributor-guide-peer-review:

				===========
				Peer Review
				===========

				Peer review is a process in which developers submit code changes to a staging
				environment, so that other peers have the opportunity to review the work before
				the code changes are submitted back to the source code management repository.

				In our development platform, this process is facilitated by `Differential`_. To
				use `Differential`_, you must install arcanist and set it up.

				.. seealso::

				* :ref:`contributor-guide-setup-your-development-environment`

				In Kolab development, this stage allows work to progress on writing tests that
				fail, first, subsequently supplemented by the work on code changes that make
				those tests succeed.

				.. seealso::

				* :ref:`contributor-guide-test-driven-development`

				The changes submitted are reviewed on Thursday afternoons at the latest.

				Your changes need to be reviewed by at least one other person, who is a
				software development project member.

				In :ref:`contributor-guide-test-driven-development`, the submission of the
				differential associated with your review process aides in the staging of the
				code changes.