Currently the PRProcessor sets labels based on the status. These are:
- Waiting on contributor
- Needs testing
- Needs re-review
A lot of this comes from before Github had a proper review flow so I want to open the discussion on how to deal with this.
My impression is that nobody uses the
Needs testing so I’m proposing to drop that without replacement.
- Keep Needs testing label
- Drop Needs testing label
The Waiting on contributor and Not yet reviewed / Needs re-review labels can be useful in filtering. However, Github can also filter on no reviews. It can also dismiss reviews when new code is pushed (enabled on foreman.git). That means you already have a filtering mechanism.
My proposal is to enable dismissing reviews on push for all repositories and dropping the labels
- Enable dismissing stale reviews on all repositories and drop labels
- Enable dismissing stale reviews on all repositories but keep labels
- Keep labels