From e3d6cb0724a413a2279cc9e08b31f8adf2acec44 Mon Sep 17 00:00:00 2001 From: Simon Westphahl Date: Wed, 1 Mar 2023 14:03:14 +0100 Subject: Don't add PR title in commit message on squash Github will use the PR title as the commit subject for squash merges, so we don't need include the title once again in the commit description. Change-Id: Id5a00701c236235f5a49abd025bcfad1b2da916c --- tests/fakegithub.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests/fakegithub.py') diff --git a/tests/fakegithub.py b/tests/fakegithub.py index 725c083e2..25dcb15da 100644 --- a/tests/fakegithub.py +++ b/tests/fakegithub.py @@ -730,7 +730,7 @@ class FakeGithubSession(object): 'message': 'Merge not allowed because of fake reason', } return FakeResponse(data, 405, 'Method not allowed') - pr.setMerged(json["commit_message"]) + pr.setMerged(json.get("commit_message", "")) return FakeResponse({"merged": True}, 200) return FakeResponse(None, 404) -- cgit v1.2.1