commit | 3068f16ed51e18ba4214be01da96f5bb9b22ca38 | [log] [tgz] |
---|---|---|
author | Edwin Kempin <ekempin@google.com> | Fri Apr 16 11:16:23 2021 +0200 |
committer | Edwin Kempin <ekempin@google.com> | Fri Apr 16 16:15:30 2021 +0200 |
tree | 7ff42c06e687ee9ec17830a2095b138c7e04ae6f | |
parent | f93707bfe94d22afc6b16c35b7e35b50ce95b2cf [diff] |
CodeOwnerApprovalCheck: Avoid passing implicit approval flag into all methods This was a bit confusing: * if enableImplicitApprovalFromUploader == true, then patchSetUploader was guaranteed to be non-null * if enableImplicitApprovalFromUploader == false, then patchSetUploader could have been null (e.g. for calls done from getFileStatusesForAccount(...)) Instead of passing in enableImplicitApprovalFromUploader and the patchSetUploader, just pass in a nullable implicitApprover (account ID of the implicit approver). If implicitApprover is set, there is an implicit approval from that account. If implicitApprover is not set there is no implicit approval. That's easier to understand. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: Ia1f102d5e9621833cb8b9723fed658cec83698fd
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.