Merge lp:~asbalderson/jenkins-launchpad-plugin/trigger-fix into lp:jenkins-launchpad-plugin

Proposed by Alexander Balderson
Status: Merged
Approved by: Joshua Powers
Approved revision: 147
Merged at revision: 147
Proposed branch: lp:~asbalderson/jenkins-launchpad-plugin/trigger-fix
Merge into: lp:jenkins-launchpad-plugin
Diff against target: 12 lines (+1/-1)
1 file modified
jlp/jenkinsutils.py (+1/-1)
To merge this branch: bzr merge lp:~asbalderson/jenkins-launchpad-plugin/trigger-fix
Reviewer Review Type Date Requested Status
Joshua Powers (community) Approve
Review via email: mp+368097@code.launchpad.net

Commit message

Remove revision arg from autoland no commit message arguments

Revision is not used in any of templates used when autoland
commit message is empty.

To post a comment you must log in.
Revision history for this message
Joshua Powers (powersj) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'jlp/jenkinsutils.py'
--- jlp/jenkinsutils.py 2019-04-24 17:57:35 +0000
+++ jlp/jenkinsutils.py 2019-05-29 21:15:11 +0000
@@ -832,7 +832,7 @@
832 'autolanding for "%s".' % (mp.web_link))832 'autolanding for "%s".' % (mp.web_link))
833 template_args = {833 template_args = {
834 'result': 'FAILED',834 'result': 'FAILED',
835 'revision': mp.source_branch.revision_count,835 'revision': None,
836 'build_url': None,836 'build_url': None,
837 'merge_url': mp.web_link837 'merge_url': mp.web_link
838 }838 }

Subscribers

People subscribed via source and target branches