Demo
Indicators
Each code review has an indicator for whether it was the Pull Request’s first review.First to review.
< 2 hours.
> 2 hours.
What ready means
What ready means
The timestamp used for being “ready” is calculated based on what is available first:
While this may seem overly complex, it’s important so that the time elapsed calculation is fair for developers.
- The timestamp of the first review request.
- Or the timestamp when the Pull Request was opened. A draft isn’t considered open.
- Or the timestamp the Pull Request was created.
Examples
Examples
- A Pull Request that got reviewed while in draft state, without requesting reviews, would use the Pull Request creation time.
- A Pull Request that got created in draft, converted to open, and then got reviewed without ever requesting reviews, would use the timestamp of when it was converted to open.
- A Pull Request that got created in draft, and requested multiple reviewers, would use the timestamp of the first review request.
Weekends are not included when calculating time elapsed.
Use cases
- Easily see developer’s code reviews in one place. Quickly open the pull requests and go through the comments.
- Spot LGTM stamps in a glance (number of comments vs Pull Request size).
- Coach a developer into becoming a better reviewer.
- Assess their communication style.
- Assess the depth of their reviews.
- Assess review frequency.
Ready to unlock continuous improvement in your team?