diff options
author | Fredrik Luthander <fredrik.luthander@sonyericsson.com> | 2011-11-10 00:15:09 +0100 |
---|---|---|
committer | Martin Fick <mfick@codeaurora.org> | 2011-12-28 12:34:24 -0700 |
commit | 1898adb8c0daad1adf1e39abd169216d0bd496d0 (patch) | |
tree | f35c71f154944a8c4989c4f2c7750044c929f2ba | |
parent | d996088829dc80235efbb5f33b1c8bb29f7a9f00 (diff) |
Access control documentation: Verify
Updated some text in the Per project-section and edited the verified
section to reflect the current label.
Change-Id: I6460cb7a9e95d67730e3bc70e79779e27bdf815f
Signed-off-by: Fredrik Luthander <fredrik.luthander@sonyericsson.com>
-rw-r--r-- | Documentation/access-control.txt | 29 |
1 files changed, 18 insertions, 11 deletions
diff --git a/Documentation/access-control.txt b/Documentation/access-control.txt index 6aec6520e0..5aa1cd679c 100644 --- a/Documentation/access-control.txt +++ b/Documentation/access-control.txt @@ -292,27 +292,34 @@ useful on the `Read Access` category when granting `-1 No Access` within a specific project to deny access to a group. -Categories ----------- +[[access_category]] +Access Categories +----------------- Gerrit comes pre-configured with several default categories that can be granted to groups within projects, enabling functionality for that group's members. -[[category_VRIF]] -Verified -~~~~~~~~ -The verified category can have any meaning the project desires. -It was originally invented by the Android Open Source Project to -mean 'compiles, passes basic unit tests'. +[[category_label-Verified]] +Label: Verified +~~~~~~~~~~~~~~~ + +The verified category is one of two default categories that is +configured upon the creation of a Gerrit instance. It may have +any meaning the project desires. It was originally invented by +the Android Open Source Project to mean +'compiles, passes basic unit tests'. The range of values is: * -1 Fails + Tried to compile, but got a compile error, or tried to run tests, -but one or more tests did not pass. +but one or more tests did not pass. This value is valid +across all patch sets in the same change, i.e. the reviewer must +actively change his/her review to something else before the change +is submittable. + *Any -1 blocks submit.* @@ -326,8 +333,8 @@ Compiled (and ran tests) successfully. + *Any +1 enables submit.* -In order to submit a change, the change must have a `+1 Verified` in -this category from at least one authorized user, and no `-1 Fails` +For a change to be submittable, the change must have a `+1 Verified` +in this category from at least one authorized user, and no `-1 Fails` from an authorized user. Thus, `-1 Fails` can block a submit, while `+1 Verified` enables a submit. |