tree a8d7263139d90bb72673c1f852bd486d9df929d9
parent f9dbc65e0ffa3449a88b3e2567b640a41766892c
author Alex Rakoczy <alex@golang.org> 1657742833 -0400
committer Gopher Robot <gobot@golang.org> 1658260817 +0000

cmd/gopherbot: omit owner from assign reviewers

Assign reviewers correctly omits the owner from the list of gerrit
reviewers, but not from the list of metadata on the CL. When adding
a "Run-TryBot +1" vote to a CL, the owner is listed as a Reviewer in the
metadata.

This change updates both cases to use the same filter, simplifying the
humanReviewersOnChange to check for bots and owner the same way for
metadata and reviewers.

Fixes golang/go#53077

Change-Id: Ia3d4939f6fb8d81cfb371683025b5e3ce0282fd7
Reviewed-on: https://go-review.googlesource.com/c/build/+/417479
Run-TryBot: Jenny Rakoczy <jenny@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Jenny Rakoczy <jenny@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>