commit | 1f37e09c2383a9fa6cc2970e749b1195dd08c663 | [log] [tgz] |
---|---|---|
author | Edwin Kempin <ekempin@google.com> | Thu Jan 20 14:14:24 2022 +0100 |
committer | Edwin Kempin <ekempin@google.com> | Thu Jan 20 14:18:49 2022 +0100 |
tree | 4de693a97b13389417d5aaaf3916c692ddd8f1e7 | |
parent | 241ad38e52f2f270b096fa4d561c2e2c7b9cd27c [diff] |
Fix NPE if OWNERS file imports '/' instead of '/OWNERS' An import in an OWNERS file must specify the file name of the imported OWNERS file. This means to import the OWNERS file from the root directory '/OWNERS' must be imported. Trying to attempt importing the root folder '/' failed with a NPE because the file name of the path is null in this case. Fix the NPE and return a proper error message in this case. Signed-off-by: Edwin Kempin <ekempin@google.com> Change-Id: Ib92dc40e007ce7a9693896cf394d57adc844db07
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.
For testing the plugin with Gerrit FE Dev Helper build the JavaScript bundle and copy it to the plugins/
folder:
bazel build //plugins/code-owners/ui:code-owners cp -f bazel-bin/plugins/code-owners/ui/code-owners.js plugins/
and let the Dev Helper redirect from .+/plugins/code-owners/static/code-owners.js
to http://localhost:8081/plugins_/code-owners.js
.