-
Notifications
You must be signed in to change notification settings - Fork 8
Description
The in-review
label is reserved for the team member / QA reviewing the Pull Request code/change. The reviewer applies the label to communicate with the rest of the team that they are actively reviewing the PR and to avoid duplication of effort.
We used to require reviewers to write a comment on the PR (typically a π :eyes:
emoji) to indicate that they were reviewing the PR, but we decided that using a label instead of a comment was clearer (especially to non-technical team members/stakeholders) and worked well for us.
only the reviewer should apply the
in-review
label. this will start the clock on their Timer. π
Process Automation enhancement
help wanted
Once the pull request has been merged @dwylbot π€ will remove the in-review
label from the Pull Request and apply the please-test
label. π
Additionally/Optionally @dwylbot π€ will parse the commit messages in the PR for any issue numbers and apply the please-test
label to those issues too. β¨