Difference between revisions of "Code Review Workflow"
From Agility
| (4 intermediate revisions by the same user not shown) | |||
| Line 1: | Line 1: | ||
| − | < | + | <ol> |
<li>We are using Crucible software for code review and also for general review of non svn tasks (if required)</li> | <li>We are using Crucible software for code review and also for general review of non svn tasks (if required)</li> | ||
<li>Our workflow is based on [https://confluence.atlassian.com/crucible/the-crucible-workflow-298977485.html One-to-One Reviews] (without moderator). | <li>Our workflow is based on [https://confluence.atlassian.com/crucible/the-crucible-workflow-298977485.html One-to-One Reviews] (without moderator). | ||
| − | [[File:onetooneworkflow.png]]</li> | + | [[File:onetooneworkflow.png|none]]</li> |
| − | <li>Responsibility of task owner ( | + | <li>Responsibility of task owner (Assignee) is to create review and assign to it Reviewer</li> |
<li>There is no general rule for Reviewer selection it is agreement between Task Owner and Reviewer (agreement can be made on Daily)</li> | <li>There is no general rule for Reviewer selection it is agreement between Task Owner and Reviewer (agreement can be made on Daily)</li> | ||
<li>After positive review (or settlement with Reviewer ;-) : | <li>After positive review (or settlement with Reviewer ;-) : | ||
*Reviewer complete review | *Reviewer complete review | ||
| − | * | + | *Task owner close review |
| − | </ | + | </ol> |
Latest revision as of 07:26, 13 October 2017
- We are using Crucible software for code review and also for general review of non svn tasks (if required)
- Our workflow is based on One-to-One Reviews (without moderator).
- Responsibility of task owner (Assignee) is to create review and assign to it Reviewer
- There is no general rule for Reviewer selection it is agreement between Task Owner and Reviewer (agreement can be made on Daily)
- After positive review (or settlement with Reviewer ;-) :
- Reviewer complete review
- Task owner close review
