Better communication and expectation management around review turnaround time
Objective
Address the expectation gap that MR authors have when waiting on a review.
Action
Require reviewers to indicate whether they are not able to get to a review within 48 hours.
Implementation
- Update the code review documentation to indicate that reviewers assigned to an MR should leave a comment on the MR to indicate if they expect that they might not be able to get to the review within 48hrs. Bonus points for providing an expected date they can get to it.
- Ensure the change is communicated to all of engineering
- Add to week-in-review document
- Consider whether this change warrants an acknowledgment receipt
Effort
Impact: Medium - while it might not shorten the review cycle the managing of expectation will have an overall positive impact
Implementation effort: Low