commit | 659120854b5ef97ec9ec28dd1f5f4a3ed13bdeb0 | [log] [tgz] |
---|---|---|
author | Luca Milanesio <luca.milanesio@gmail.com> | Thu Sep 30 23:21:44 2021 +0100 |
committer | Luca Milanesio <luca.milanesio@gmail.com> | Thu Sep 30 23:23:08 2021 +0100 |
tree | dd134c825067f9078a83671a0ebaf1a30664d812 | |
parent | 37adbe296dc60f8fdf87251589b9a7a82d2c721f [diff] |
Fix owners-autoassign ReviewerState field type in project config The ReviewerState was rendered as a general string in the projecyt config screen making it error-prone. Change the configuration entry type to a LIST so that it becomes clear what are the only allowed values. Change-Id: I3f1e4468fda030559c6c094ea7b750d30a837b89
diff --git a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/AutoassignConfigModule.java b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/AutoassignConfigModule.java index 5536ebc..dfa812d 100644 --- a/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/AutoassignConfigModule.java +++ b/owners-autoassign/src/main/java/com/googlesource/gerrit/owners/common/AutoassignConfigModule.java
@@ -40,7 +40,7 @@ new ProjectConfigEntry( "Auto-assign field", ReviewerState.REVIEWER.name(), - ProjectConfigEntryType.STRING, + ProjectConfigEntryType.LIST, Arrays.asList(ReviewerState.CC.name(), ReviewerState.REVIEWER.name()), true, "Change field to use for the assigned accounts"));