Remove redundant predicates from copy conditions

If "is:ANY" is matched, all other OR-conditions can be dropped, since
"is:ANY" already matches all approvals [1]

Removing the redundant predicates makes the copy conditions easier
to grasp. This is more important now that copy conditions are included
into change messages, and hence exposed to all users (see b/234821250).

[1] https://gerrit-review.googlesource.com/Documentation/config-labels.html#is_any

Change-Id: If3aad1a281902574f4cd3e048e9b921988ba11ff
Reviewed-on: https://go-review.googlesource.com/c/All-Projects/+/416054
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>

project.config

diff --git a/project.config b/project.config
index 21d73db..1dbca7c 100644
--- a/project.config
+++ b/project.config
@@ -85,14 +85,14 @@
 [label "Code-Review"]
 	function = NoBlock
 	defaultValue = 0
-	copyCondition = changekind:NO_CHANGE OR is:ANY
+	copyCondition = is:ANY
 	value = 0 No score
 	value = +1 Looks good to me, but someone else must approve
 	value = +2 Looks good to me, approved
 [label "Hold"]
 	function = NoBlock
 	defaultValue = 0
-	copyCondition = changekind:NO_CHANGE OR is:ANY
+	copyCondition = is:ANY
 	value = 0 This change can be submitted.
 	value = +1 Hold this change: don't submit it yet.
 [label "Run-TryBot"]