Make ignoreReviewers exclude users from Assignment or add excludeReviewers option

[frq-asgard-josi]

Tell us more.

Examples:

• When using reviewersFromCodeowners I want to exclude certain users (e.g. renovate-bot) from the resolved list of users.
• When specifiying reviewers in a group level config preset, one might want to exclude certain users from individual projects.

While one might think ignoreReviewers can be used for this, this is not the case at the moment. This is correctly documented on the configuration options page for ignoreReviewers but reading the Assignees and Reviewers section in the automerge docs leaves enough room for interpretation to assume it might also exclude these users from actual assignment.

To solve this we could either extend the ignoreReviewers behaviour or add a new config option (such as excludeReviewers)

If this request gets approved I would offer to create a PR for this as well.

[RahulGautamSingh]

Sounds good to me.

[frq-asgard-josi]

Which approach would you prefer (extend ignoreReviewers or add new excludeReviewers)?
Also, what Actions do I need to take to proceed?

[RahulGautamSingh]

@jamietanna @viceice I prefer to create a new config option, ignoreReviewers is tied to automerge feature so I prefer the excludeReviewers approach.

thoughts?

[viceice]

sounds plausible to add as there's no other option to exclude/remove code owner users from reviewer list. 🤔

[frq-asgard-josi]

What do I have to do to continue with this?

[RahulGautamSingh]

create a PR