Disable validation of OWNERS files on submit by default

Changes to OWNERS files are already validated on upload / commit
creation, hence there is no urgent need to validate them once again on
submit. This means we can skip the validation on submit by default.

There are reasons for validating OWNERS files on submit:

1. if the validation of OWNERS files on upload / commit creation is
   switched off (e.g. because it is preferred to report issues with
   OWNERS files as findings rather than rejecting the upload / commit
   creation)
2. if changes stay open for a long time it can happen that changes to
   OWNERS files that were valid on upload are invalid on submit (e.g. a
   newly added import was resolvable on upload, but then the imported
   OWNERS file got deleted and it's no longer available when the submit
   is done)

1. is not the case by default and 2. happens rarely enough so that by
default we do not need to do the validation on submit.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: Iba334f1a92b7f0e556855982198de9a52eefada0
7 files changed
tree: a0e6694b90899513c9aae188f1485722527f5e6e
  1. java/
  2. javatests/
  3. proto/
  4. resources/
  5. test/
  6. ui/
  7. .eslintrc.json
  8. .gitignore
  9. .gitreview
  10. bower.json
  11. BUILD
  12. LICENSE
  13. package-lock.json
  14. package.json
  15. README.md
README.md

Gerrit Code Review code-owners plugin

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.