commit | 06e0a6422285d7f6fca45db676168644a461946c | [log] [tgz] |
---|---|---|
author | Edwin Kempin <ekempin@google.com> | Thu May 06 10:16:45 2021 +0000 |
committer | Edwin Kempin <ekempin@google.com> | Thu May 06 11:50:00 2021 +0000 |
tree | 9a6a3d5a7c45f228f3cb53f3d50a227605f03f90 | |
parent | 0de334fe527a3825bd81a0a47c537a14d5889251 [diff] |
CodeOwnerResolver: Use ImmutableList.Builder<String> to collect messages CodeOwnerResolver consists out of several steps to resolve code owner emails to code owner accounts. For each step we collect debug messages so that issues with resolving code owners can easily be investigated. So far the internal methods returned OptionalResultWithMessages which is the result of the method plus collected debug messages. Going forward we want to refactor CodeOwnerResolver to process all emails at once rather than each email one by one. For this most of the steps will be converted into operations on a stream, e.g. the methods will filter the stream or map entries in the stream. When we do this the methods can no longer return messages as part of the result. To enable the refactoring, this change changes the internal methods to no longer return OptionalResultWithMessages, but instead return the plain result and get an ImmutableList.Builder<String> passed in that is used to collect messages. This also simplifies some of the code. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: Idf625ecf1784553dd6daa24a7d4862532e2524ac
This plugin provides support for defining code owners for files in a repository.
If the code-owners plugin is enabled, changes can only be submitted if all touched files are covered by approvals from code owners.
Also see resources/Documentation/about.md.
IMPORTANT: Before installing/enabling the plugin follow the instructions from the setup guide, see resources/Documentation/setup-guide.md.