diff --git a/gcl.py b/gcl.py index 56655c294..ce6edabab 100755 --- a/gcl.py +++ b/gcl.py @@ -1045,7 +1045,8 @@ def CMDcommit(change_info, args): elif revision: change_info.description += "\nCommitted: " + revision change_info.CloseIssue() - props = change_info.RpcServer().get_issue_properties(change_info.issue) + props = change_info.RpcServer().get_issue_properties( + change_info.issue, False) patch_num = len(props['patchsets']) comment = "Committed patchset #%d manually as r%s" % (patch_num, revision) comment += ' (presubmit successful).' if not bypassed else '.' diff --git a/git_cl.py b/git_cl.py index e515f89b0..11266914a 100755 --- a/git_cl.py +++ b/git_cl.py @@ -1495,7 +1495,7 @@ def SendUpstream(parser, args, cmd): print ('Closing issue ' '(you may be prompted for your codereview password)...') cl.CloseIssue() - props = cl.RpcServer().get_issue_properties(cl.GetIssue()) + props = cl.RpcServer().get_issue_properties(cl.GetIssue(), False) patch_num = len(props['patchset']) comment = "Committed patchset #%d manually as r%s" % (patch_num, revision) comment += ' (presubmit successful).' if not options.bypass_hooks else '.' diff --git a/tests/gcl_unittest.py b/tests/gcl_unittest.py index 77cc56fb2..ddaab3a82 100755 --- a/tests/gcl_unittest.py +++ b/tests/gcl_unittest.py @@ -59,7 +59,8 @@ class GclTestsBase(SuperMoxTestBase): change_info._comments_added = [] class RpcServer(object): - def get_issue_properties(self, *_): # pylint: disable=R0201 + # pylint: disable=R0201,W0613 + def get_issue_properties(self, issue, messages): return { 'patchsets': [1337] } change_info.RpcServer = RpcServer