commit | 1565aa6568dc9398033ec9884c70dbe62516d57b | [log] [tgz] |
---|---|---|
author | Chris Poucet <poucet@google.com> | Mon Nov 21 11:46:47 2022 +0100 |
committer | Chris Poucet <poucet@google.com> | Mon Nov 21 12:00:30 2022 +0100 |
tree | 62cc7e282256a3c0d6978a68dbb31d732b7a5838 | |
parent | dc93a41c4ba9e775f016450eb10ab4b00ce38c47 [diff] |
Show files as unknown when their code-owner status is not present The absence of files in code-owners needn't be an error as originally coded. It could be due to the fact that the files are not part of the change list (if a file is being commented on that's not part of the change). Or it could be files from a merge commit. Instead show their status as not needing code-owners approval. Screenshot: (Before) https://imgur.com/a/qf47aAj, (After) https://imgur.com/a/J3PuGGU Google-Bug-Id: b/228404313, b/253451848 Change-Id: I73d1c2b7c06714d1ac438a66bbf6fc9389fd2238
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
.