Convert notify parameter inside gerrit_util
Bug: skia:6744
Change-Id: I0f84bd685bfacf692190f8636208122cf88d0b82
Reviewed-on: https://chromium-review.googlesource.com/529016
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Aaron Gable <agable@chromium.org>
diff --git a/git_cl.py b/git_cl.py
index 8e8775b..6fbaa15 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -3105,8 +3105,7 @@
print('Adding self-LGTM (Code-Review +1) because of TBRs.')
gerrit_util.SetReview(
self._GetGerritHost(), self.GetIssue(),
- labels={'Code-Review': 1},
- notify='ALL' if options.send_mail else 'NONE')
+ labels={'Code-Review': 1}, notify=bool(options.send_mail))
return 0
@@ -3194,7 +3193,7 @@
kwargs = {'labels': {'Commit-Queue': vote_map[new_state]}}
if new_state == _CQState.DRY_RUN:
# Don't spam everybody reviewer/owner.
- kwargs['notify'] = 'NONE'
+ kwargs['notify'] = False
gerrit_util.SetReview(self._GetGerritHost(), self.GetIssue(), **kwargs)
def CannotTriggerTryJobReason(self):