Consider only emails that belong to multiple active accounts as ambiguous Code owner emails that belong to multiple accounts are detected as ambiguous and their code ownership is ignored. This is intended however if an email belongs to multiple accounts and only of them is active, that's fine as there is exactly one active account to which the code ownership is assigned. For CodeOwnerResolver this means that it has to load the accounts, and check if they are active, before deciding if an email is ambiguous. Doing this changes, actually improves, the debug logs in case there is an orphaned email. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: I50f06e4e055ee04ac17c8873f0e08c0d6aca4b5e
diff --git a/resources/Documentation/validation.md b/resources/Documentation/validation.md index 3b0dc95..69929f4 100644 --- a/resources/Documentation/validation.md +++ b/resources/Documentation/validation.md
@@ -102,7 +102,7 @@ setting) * it is a non-visible secondary email * there is no account that has this email assigned - * it is ambiguous (the same email is assigned to multiple accounts) + * it is ambiguous (the same email is assigned to multiple active accounts) * it has an email domain that is disallowed (see [allowedEmailDomain](config.html#pluginCodeOwnersAllowedEmailDomain)) configuration