commit | 847bee6c18d98cd290eeb5198586fb1167c59c98 | [log] [tgz] |
---|---|---|
author | Kamil Musin <kamilm@google.com> | Mon Aug 28 11:53:28 2023 +0200 |
committer | Kamil Musin <kamilm@google.com> | Mon Aug 28 11:53:28 2023 +0200 |
tree | 7fb97cc0be4d9f5c5ab8d69b451ace09ba0a7ddd | |
parent | 8a9288f3f50a02044c962db604999be8b8626c1e [diff] |
Set reasons to undefined if the path is not in Owners response. codeOwnerStatusMap.get can in fact return `undefined`. This is in particular evident, by `if (statusItem === undefined)` in extractStatus. Fix type of the argument in extractStatus, and add "?" when accessing reasons. Change-Id: I141004c77e7c5f048de7a552d0e65c37566dd682
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.
For a detailed description of the plugin functionality please refer to the plugin documentation.
IMPORTANT: Before installing/enabling the plugin follow the instructions from the setup guide.
NOTE: The plugin documentation only renders correctly when the plugin is installed in Gerrit and the documentation is accessed via https:///plugins/code-owners/Documentation/index.html. If you want to read the documentation before installing the plugin, you can find it properly rendered here.
From the root of the gerrit repository.
bazel test //plugins/code-owners/web:karma_test
For testing the plugin with the Gerrit FE Dev Helper the command below builds
bazel build //plugins/code-owners/web:code-owners ln -s bazel-bin/plugins/code-owners/web/code-owners.js polygerrit-ui/app/plugins/
and let the Dev Helper redirect from .+/plugins/code-owners/static/code-owners.js
to http://localhost:8081/plugins/code-owners.js
.