commit | d6f24b9c39670ef12625a4ed6a9d886f087bca51 | [log] [tgz] |
---|---|---|
author | Edwin Kempin <ekempin@google.com> | Wed Oct 14 12:46:54 2020 +0200 |
committer | Edwin Kempin <ekempin@google.com> | Wed Oct 14 13:04:41 2020 +0200 |
tree | 120d1b84d163f353fbcac065bbd8ac000b023b4d | |
parent | fde83ef6d7a92553317ad96bdee55f8487a48982 [diff] |
GetCodeOwnerProjectConfig: If disabled set only status.disabled If the code owners functionality is disabled for the project (status.disabled = true) then the other config parameters are not relevant and do not need to be returned. Trying to return them anyway is bad, because they may be invalid and then the request cannot succeed. E.g. at the moment GetCodeOwnerProjectConfig fails with 500 Internal Server Error if a non-existing label is configured as required approval. The required approval configuration is not relevant while the code owners functionality is disabled and should not cause the request to fail. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: Ifd2cce2ce973a35b12dc855b8c074d0a3d981a62
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