opendev/elastic-recheck
Code Issues Proposed changes

Don't encourage reverify

Browse Source 
Reverify makes the gate cry. Let's not encourage it.

Change-Id: I601a31a8899c6752cb8ba69a8fe87a2d8d0742a6
This commit is contained in:
Michael Still
2014-02-03 10:36:59 +11:00
parent 0170e4f97f
commit 9bfff2bbd6
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
elastic_recheck/elasticRecheck.py
View File

@@ -265,13 +265,8 @@ doing that manually if someone hasn't already. For a code review
which is not yet approved, you can recheck by leaving a code which is not yet approved, you can recheck by leaving a code
review comment with just the text: review comment with just the text:
recheck bug %(bug)s recheck bug %(bug)s""" % {'bugs': "\n- ".join(bug_urls),
'bug': list(event.bugs)[0]}
For a code review which has been approved but failed to merge,
you can reverify by leaving a comment like this:
reverify bug %(bug)s""" % {'bugs': "\n- ".join(bug_urls),
'bug': list(event.bugs)[0]}
else: else:
message = ("I noticed tempest failed, refer to: " message = ("I noticed tempest failed, refer to: "
"https://wiki.openstack.org/wiki/" "https://wiki.openstack.org/wiki/"