Changeset View
Changeset View
Standalone View
Standalone View
src/docs/user/userguide/differential.diviner
@title Differential User Guide | @title Differential User Guide | ||||
@group userguide | @group userguide | ||||
Guide to the Differential (pre-push code review) tool and workflow. | Guide to the Differential (pre-push code review) tool and workflow. | ||||
= Overview = | = Overview = | ||||
Phabricator supports two code review workflows, "review" (pre-push) and | Phorge supports two code review workflows, "review" (pre-push) and | ||||
"audit" (post-push). To understand the differences between the two, see | "audit" (post-push). To understand the differences between the two, see | ||||
@{article:User Guide: Review vs Audit}. | @{article:User Guide: Review vs Audit}. | ||||
This document summarizes the pre-push "review" workflow implemented by the tool | This document summarizes the pre-push "review" workflow implemented by the tool | ||||
//Differential//. | //Differential//. | ||||
= How Review Works = | = How Review Works = | ||||
Code review in Phabricator is a lightweight, asynchronous web-based process. If | Code review in Phorge is a lightweight, asynchronous web-based process. If | ||||
you are familiar with GitHub, it is similar to how pull requests work: | you are familiar with GitHub, it is similar to how pull requests work: | ||||
- An author prepares a change to a codebase, then sends it for review. They | - An author prepares a change to a codebase, then sends it for review. They | ||||
specify who they want to review it (additional users may be notified as | specify who they want to review it (additional users may be notified as | ||||
well, see below). The change itself is called a "Differential Revision". | well, see below). The change itself is called a "Differential Revision". | ||||
- The reviewers receive an email asking them to review the change. | - The reviewers receive an email asking them to review the change. | ||||
- The reviewers inspect the change and either discuss it, approve it, or | - The reviewers inspect the change and either discuss it, approve it, or | ||||
request changes (e.g., if they identify problems or bugs). | request changes (e.g., if they identify problems or bugs). | ||||
▲ Show 20 Lines • Show All 46 Lines • Show Last 20 Lines |
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0