update owners documentation to match the current plugin
BUG=1243480
Change-Id: I5a3fec78f6cabcef558523d84b0f2d3d31b83b44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3124171
Reviewed-by: David Dorwin <[email protected]>
Commit-Queue: Anthony Polito <[email protected]>
Cr-Commit-Position: refs/heads/main@{#969554}
diff --git a/docs/code_reviews.md b/docs/code_reviews.md
index d32e612..aa3580ec1 100644
--- a/docs/code_reviews.md
+++ b/docs/code_reviews.md
@@ -124,7 +124,7 @@
### OWNERS file details
-Refer to the [source code](https://chromium.googlesource.com/chromium/tools/depot_tools/+/main/owners.py)
+Refer to the [owners plugin](https://github.com/GerritCodeReview/plugins_code-owners/blob/master/resources/Documentation/backend-find-owners.md)
for all details on the file format.
This example indicates that two people are owners, in addition to any owners
@@ -171,19 +171,15 @@
per-file readme.txt=*
```
-Note that `per-file` directives cannot directly specify subdirectories, e.g:
-```
-per-file foo/[email protected]
-```
-
-is not OK; instead, place a `per-file` directive in `foo/OWNERS`.
-
Other `OWNERS` files can be included by reference by listing the path to the
file with `file://...`. This example indicates that only the people listed in
`//ipc/SECURITY_OWNERS` can review the messages files:
```
per-file *_messages*.h=set noparent
per-file *_messages*.h=file://ipc/SECURITY_OWNERS
+
+File globbing is supported using the
+[simple path expression format](https://github.com/GerritCodeReview/plugins_code-owners/blob/master/resources/Documentation/path-expressions.md#simplePathExpressions)
```
### Owners-Override
