Page MenuHomePhabricator

Permit Herald to remove reviewer
Closed, DuplicatePublic

Description

There are certain projects (and users) which should never be a reviewer on a diff. For example, we have a project that consists of "all committers" which is useful for policy and other reasons. However, adding 300+ people to the emails and having them show up as a reviewer is something we want to avoid.

It would be nice if we could set a Herald rule "if any of X Y OR Z are reviewers" then "remove X Y and Z as reviewers".

Event Timeline

eadler updated the task description. (Show Details)Apr 30 2015, 12:49 AM
eadler added a project: Herald.
eadler added a subscriber: eadler.
eadler created this task.
eadler raised the priority of this task from to Needs Triage.
chad added a subscriber: chad.Apr 30 2015, 12:56 AM

This doesn't seem scalable, maintaining a blocklist, that is. Is this an actual abuse case that happened?

chad added a comment.Apr 30 2015, 12:58 AM

Ah, well nevermind. haha, yeah T5602 is a slightly better solution.