tree 70a12235eacedf1f7ca826326fef031f7b58f710
parent 6c6082de2d7fe5ee49033a0a3013db40205cadd4
author Dmitri Shuralyov <dmitshur@golang.org> 1580758133 -0500
committer Dmitri Shuralyov <dmitshur@golang.org> 1580933371 +0000

devapp/owners: remove requirement of non-empty Gerrit email

Some people may not have a Gerrit account (yet, or in general).
Instead of entering a fake email, just remove the requirement
to have a non-empty Gerrit email. It's useful to have an owner
who only has a GitHub account.

The original code in CL 111295 had a len(p.Emails) == 0 check
because it was attempting to access p.Emails[0], but after
CL 192102, it is safe to use p.Gerrit directly.

Change-Id: I737d287f69f8ff7a0377f8e324bf6318d8c18d94
Reviewed-on: https://go-review.googlesource.com/c/build/+/217540
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Run-TryBot: Alexander Rakoczy <alex@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>