PR label workflow

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

0 voters

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

0 voters

So the poll has closed and the results are clear and people favor removal. This week is busy, but probably a nice Friday project. I’ll look into writing some tools to easily clean up labels and enable the branch rules.