commit 2c376aa21bef061e6529c1459eb161ce31140d7c
author Andrii Shyshkalov <tandrii@chromium.org> Fri Jun 09 15:39:37 2017 +0200
committer Commit Bot <commit-bot@chromium.org> Fri Jun 09 13:47:25 2017 +0000
tree 8e6af796e484e5e5eaa9064889e915f3dc1cbd58
parent a07816ddce78fcbf5f865a582b7895902f9bb9a2

git cl: Fix incorrect notify parameter when setting TBR.

Follow up on https://chromium-review.googlesource.com/c/527325/

R=agable@chromium.org,rmistry@chromium.org

Bug: chromium:729967
Bug: skia:6744
Change-Id: I7443298797a7c95c8ca01624e3cbf08c95e04855
Reviewed-on: https://chromium-review.googlesource.com/529246
Reviewed-by: Ravi Mistry <rmistry@chromium.org>
Commit-Queue: Andrii Shyshkalov <tandrii@chromium.org>

git_cl.py

diff --git a/git_cl.py b/git_cl.py
index 5266b5f..7ccf8e0 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -3102,7 +3102,8 @@
       print('Adding self-LGTM (Code-Review +1) because of TBRs.')
       gerrit_util.SetReview(
           self._GetGerritHost(), self.GetIssue(),
-          labels={'Code-Review': 1}, notify=bool(options.send_mail))
+          labels={'Code-Review': 1},
+          notify='ALL' if options.send_mail else 'NONE')
 
     return 0