Hi,
On 2022-07-15 16:42:03 -0700, Jacob Champion wrote:
> I started removing stale Reviewers fairly aggressively today, as
> discussed in [1], but there was some immediate feedback and I have
> paused that process for now. If you're wondering why you are no longer
> marked as reviewer on a patch, I followed the following rule: if you
> were signed up to review before June of this year, but you haven't
> interacted with the patch in this commitfest, I removed you from the
> list. If you have thoughts/comments on this approach, please share them!
I'd make it dependent on whether there have been previous rounds of feedback
or not. If somebody spent a good amount of time reviewing a patch previously,
but then didn't review the newest version in the last few weeks, it doesn't
seem useful to remove them from the CF entry. The situation is different if
somebody has signed up but not done much.
Greetings,
Andres Freund