diff options
author | Zuul <zuul@review.opendev.org> | 2023-03-22 21:52:14 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-03-22 21:52:14 +0000 |
commit | 1ba52ec37259c9f3c09cb2401f191178f136bb50 (patch) | |
tree | 7ecf8656af0859d2ce42700029c1f411e7457eda /zuul | |
parent | c0a935c92bc4a85028484ef9ec117c8251ee247e (diff) | |
parent | e3d6cb0724a413a2279cc9e08b31f8adf2acec44 (diff) | |
download | zuul-1ba52ec37259c9f3c09cb2401f191178f136bb50.tar.gz |
Merge "Don't add PR title in commit message on squash"
Diffstat (limited to 'zuul')
-rw-r--r-- | zuul/driver/github/githubreporter.py | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/zuul/driver/github/githubreporter.py b/zuul/driver/github/githubreporter.py index 1f44303bd..396516038 100644 --- a/zuul/driver/github/githubreporter.py +++ b/zuul/driver/github/githubreporter.py @@ -193,13 +193,13 @@ class GithubReporter(BaseReporter): self.log.warning('Merge mode %s not supported by Github', mode) raise MergeFailure('Merge mode %s not supported by Github' % mode) - merge_mode = self.merge_modes[merge_mode] project = item.change.project.name pr_number = item.change.number sha = item.change.patchset log.debug('Reporting change %s, params %s, merging via API', item.change, self.config) - message = self._formatMergeMessage(item.change) + message = self._formatMergeMessage(item.change, merge_mode) + merge_mode = self.merge_modes[merge_mode] for i in [1, 2]: try: @@ -319,10 +319,13 @@ class GithubReporter(BaseReporter): self.connection.unlabelPull(project, pr_number, label, zuul_event_id=item.event) - def _formatMergeMessage(self, change): + def _formatMergeMessage(self, change, merge_mode): message = [] - if change.title: - message.append(change.title) + # For squash merges we don't need to add the title to the body + # as it will already be set as the commit subject. + if merge_mode != model.MERGER_SQUASH_MERGE: + if change.title: + message.append(change.title) if change.body_text: message.append(change.body_text) merge_message = "\n\n".join(message) |