git-codereview: change default -trybot flag to use LUCI
For when it's time.
Change-Id: Ife6df8eed0a02d9a47ffca141be083aaf3b09d28
Reviewed-on: https://go-review.googlesource.com/c/review/+/523115
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
diff --git a/git-codereview/doc.go b/git-codereview/doc.go
index 0754479..5f3e0b1 100644
--- a/git-codereview/doc.go
+++ b/git-codereview/doc.go
@@ -254,7 +254,7 @@
containing data that looks like public keys. (The most common time -nokeycheck
is needed is when checking in test cases for cryptography libraries.)
-The -trybot flag sets a Run-TryBot+1 vote on any uploaded changes.
+The -trybot flag sets a Commit-Queue+1 vote on any uploaded changes.
The Go project uses this vote to start running integration tests on the CL.
During the transition between two CI systems, the environment variable
GIT_CODEREVIEW_TRYBOT can be set to one of "luci", "farmer", or "both"
diff --git a/git-codereview/mail.go b/git-codereview/mail.go
index 4bcd85c..7fe2bcf 100644
--- a/git-codereview/mail.go
+++ b/git-codereview/mail.go
@@ -50,9 +50,9 @@
var trybotVotes []string
switch os.Getenv("GIT_CODEREVIEW_TRYBOT") {
- case "luci":
+ case "", "luci":
trybotVotes = []string{"Commit-Queue+1"}
- case "", "farmer":
+ case "farmer":
trybotVotes = []string{"Run-TryBot"}
case "both":
trybotVotes = []string{"Commit-Queue+1", "Run-TryBot"}
