Reduce range of values for the Verified label
Reduce the range from [-2..+2] to [-1..+1]. The onus is on the person
triggering the CI to select the right level of tests (through the
Allow-CI +1/+2 label) depending on the patch. From Gerrit, it is
enough to know that the selected tests passed.
This removes the overhead of manually upgrading a Verified+1 into a
Verified+2 for patches that only require shallow testing
(e.g. documentation patches).
Signed-off-by: Sandrine Bailleux <sandrine.bailleux@arm.com>
Change-Id: I0a8bccf63acca32c9d4ebf03a28dd64bbb85e5c3
diff --git a/project.config b/project.config
index b8b62f5..b597e11 100644
--- a/project.config
+++ b/project.config
@@ -22,8 +22,8 @@
label-Code-Review = -1..+1 group Non-Interactive Users
submit = group trusted-firmware-a-approvers
label-IP-Review = -1..+1 group trusted-firmware-a-ip-reviewers
- label-Verified = -2..+2 group Bot Users
- label-Verified = -2..+2 group trusted-firmware-a-ci-users
+ label-Verified = -1..+1 group Bot Users
+ label-Verified = -1..+1 group trusted-firmware-a-ci-users
label-Allow-CI = +0..+2 group trusted-firmware-a-ci-users
removeReviewer = group trusted-firmware-a-ci-users
abandon = group trusted-firmware-a-approvers
@@ -35,11 +35,9 @@
[label "Verified"]
function = MaxWithBlock
- value = -2 Fails (in-depth tests)
- value = -1 Fails (shallow tests)
+ value = -1 Fails
value = 0 No score
- value = +1 Verified (shallow tests)
- value = +2 Verified (in-depth tests)
+ value = +1 Verified
copyAllScoresIfNoCodeChange = true
copyAllScoresOnTrivialRebase = true
defaultValue = 0