From 5a4ef4537e27a403bcd583fb80870c16a12e1076 Mon Sep 17 00:00:00 2001 From: Aaron Gable Date: Thu, 24 Aug 2017 13:19:56 -0700 Subject: [PATCH] Don't send email from git-cl-description The 'notify' parameter was accidentally left out in https://crrev.com/c/544118. The calling code in git_cl.py already sets notify='NONE', so this will suppress all notifications as before. Bug: 757617 Change-Id: Ib0144d644d54b194fac0c0eb2fb51a6affa3340e Reviewed-on: https://chromium-review.googlesource.com/634245 Reviewed-by: Andrii Shyshkalov Commit-Queue: Aaron Gable --- gerrit_util.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gerrit_util.py b/gerrit_util.py index ecaa57172..f13cb8159 100644 --- a/gerrit_util.py +++ b/gerrit_util.py @@ -624,7 +624,7 @@ def SetCommitMessage(host, change, description, notify='ALL'): """Updates a commit message.""" assert notify in ('ALL', 'NONE') path = 'changes/%s/message' % change - body = {'message': description} + body = {'message': description, 'notify': notify} conn = CreateHttpConn(host, path, reqtype='PUT', body=body) try: ReadHttpResponse(conn, accept_statuses=[200, 204])