rewrite reviewers help text

Fix some errors, improve wording to remove redundancy, and improve style.

PiperOrigin-RevId: 742454824
Change-Id: I2a619049d1c953b2eb0222e841c9a1e98a3bd4be

Author: vapier

docs/reference.md

@@ -3237,7 +3237,7 @@ Parameter | Description
 <span id=git.gerrit_destination.notify href=#git.gerrit_destination.notify>notify</span> | <code><a href="#string">string</a></code> or <code>NoneType</code><br><p>Type of Gerrit notify option (https://gerrit-review.googlesource.com/Documentation/user-upload.html#notify). Sends notifications by default.</p>
 <span id=git.gerrit_destination.change_id_policy href=#git.gerrit_destination.change_id_policy>change_id_policy</span> | <code><a href="#string">string</a></code><br><p>What to do in the presence or absent of Change-Id in message:<ul> <li>`'REQUIRE'`: Require that the change_id is present in the message as a valid label</li> <li>`'FAIL_IF_PRESENT'`: Fail if found in message</li> <li>`'REUSE'`: Reuse if present. Otherwise generate a new one</li> <li>`'REPLACE'`: Replace with a new one if found</li></ul></p>
 <span id=git.gerrit_destination.allow_empty_diff_patchset href=#git.gerrit_destination.allow_empty_diff_patchset>allow_empty_diff_patchset</span> | <code><a href="#bool">bool</a></code><br><p>By default Copybara will upload a new PatchSet to Gerrit without checking the previous one. If this set to false, Copybara will download current PatchSet and check the diff against the new diff.</p>
-<span id=git.gerrit_destination.reviewers href=#git.gerrit_destination.reviewers>reviewers</span> | <code>sequence</code><br><p>The list of the reviewers will be added to gerrit change reviewer listThe element in the list is: an email, for example: "[email protected]" or label for example: ${SOME_GERRIT_REVIEWER}. These are under the condition of assuming that users have registered to gerrit repos</p>
+<span id=git.gerrit_destination.reviewers href=#git.gerrit_destination.reviewers>reviewers</span> | <code>sequence</code><br><p>The list of the reviewers to add. Each element in the list is: an email (e.g. `"[email protected]"` or label (e.g.  `"${SOME_GERRIT_REVIEWER}`). These assume that users have already registered on the Gerrit host and has access to the repos.</p>
 <span id=git.gerrit_destination.cc href=#git.gerrit_destination.cc>cc</span> | <code>sequence</code><br><p>The list of the email addresses or users that will be CCed in the review. Can use labels as the `reviewers` field.</p>
 <span id=git.gerrit_destination.labels href=#git.gerrit_destination.labels>labels</span> | <code>sequence</code><br><p>The list of labels to be pushed with the change. The format is the label along with the associated value. For example: Run-Presubmit+1</p>
 <span id=git.gerrit_destination.api_checker href=#git.gerrit_destination.api_checker>api_checker</span> | <code><a href="#checker">checker</a></code> or <code>NoneType</code><br><p>A checker for the Gerrit API endpoint provided for after_migration hooks. This field is not required if the workflow hooks don't use the origin/destination endpoints.</p>

java/com/google/copybara/git/GitModule.java

@@ -2429,10 +2429,10 @@ private ImmutableSetMultimap<String, Conclusion> convertSlugToConclusion(
             named = true,
             defaultValue = "[]",
             doc =
-                "The list of the reviewers will be added to gerrit change reviewer listThe element"
-                    + " in the list is: an email, for example: \"[email protected]\" or label for"
-                    + " example: ${SOME_GERRIT_REVIEWER}. These are under the condition of"
-                    + " assuming that users have registered to gerrit repos"),
+                "The list of the reviewers to add. Each element in the list is:"
+                    + " an email (e.g. `\"[email protected]\"` or label (e.g. "
+                    + " `\"${SOME_GERRIT_REVIEWER}`). These assume that users have already"
+                    + " registered on the Gerrit host and has access to the repos."),
         @Param(
             name = "cc",
             named = true,
@@ -2523,16 +2523,16 @@ private ImmutableSetMultimap<String, Conclusion> convertSlugToConclusion(
                     + "is used, that checker will only apply to API calls.",
             named = true,
             positional = false),
-          @Param(
-              name = "credentials",
-              allowedTypes = {
-                  @ParamType(type = UsernamePasswordIssuer.class),
-                  @ParamType(type = NoneType.class),
-              },
-              defaultValue = "None",
-              named = true,
-              positional = false,
-              doc = CREDENTIAL_DOC)
+        @Param(
+            name = "credentials",
+            allowedTypes = {
+              @ParamType(type = UsernamePasswordIssuer.class),
+              @ParamType(type = NoneType.class),
+            },
+            defaultValue = "None",
+            named = true,
+            positional = false,
+            doc = CREDENTIAL_DOC)
       },
       useStarlarkThread = true)
   @UsesFlags({GitDestinationOptions.class, GerritOptions.class})