Changeset View
Changeset View
Standalone View
Standalone View
src/docs/user/userguide/reviews_vs_audit.diviner
@title User Guide: Review vs Audit | @title User Guide: Review vs Audit | ||||
@group userguide | @group userguide | ||||
Discusses the differences between "review" and "audit" workflows. | Discusses the differences between "review" and "audit" workflows. | ||||
Overview | Overview | ||||
======== | ======== | ||||
Phabricator supports two similar but separate code review workflows: "review" | Phorge supports two similar but separate code review workflows: "review" | ||||
and "audit". | and "audit". | ||||
Review occurs in **Differential**, before changes are published. You can learn | Review occurs in **Differential**, before changes are published. You can learn | ||||
more in @{article:Differential User Guide}. | more in @{article:Differential User Guide}. | ||||
Audit occurs in **Diffusion**, after changes are published. You can learn more | Audit occurs in **Diffusion**, after changes are published. You can learn more | ||||
in @{article:Audit User Guide}. | in @{article:Audit User Guide}. | ||||
▲ Show 20 Lines • Show All 88 Lines • ▼ Show 20 Lines | |||||
Advantages of Audit | Advantages of Audit | ||||
=================== | =================== | ||||
Post-publish audit is a less powerful workflow than pre-publish review, but can | Post-publish audit is a less powerful workflow than pre-publish review, but can | ||||
supplement review and is better than nothing on its own. If you are unpersuaded | supplement review and is better than nothing on its own. If you are unpersuaded | ||||
by the arguments above (or work on a team that is unswayed), audits provide | by the arguments above (or work on a team that is unswayed), audits provide | ||||
some of the benefits of review with less friction: | some of the benefits of review with less friction: | ||||
- Audits are driven entirely by Phabricator: users do not need to install | - Audits are driven entirely by Phorge: users do not need to install | ||||
`arc`. | `arc`. | ||||
- Audits require little adjustment to existing workflows and little training. | - Audits require little adjustment to existing workflows and little training. | ||||
- Audits are completely nonblocking, and send fewer notifications than review. | - Audits are completely nonblocking, and send fewer notifications than review. | ||||
- Even if you have review, audits can be useful as a supplement to keep tabs | - Even if you have review, audits can be useful as a supplement to keep tabs | ||||
on lower-importance changes or raise issues that are discovered after | on lower-importance changes or raise issues that are discovered after | ||||
review. | review. | ||||
Recommendations | Recommendations | ||||
Show All 21 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