Change in gerrit-admin[master]: Fix 'NoneType' in is_draft argument
From Shlomo Ben David <sbendavi(a)redhat.com>:
Shlomo Ben David has submitted this change and it was merged.
Change subject: Fix 'NoneType' in is_draft argument
......................................................................
Fix 'NoneType' in is_draft argument
When merging patches they are with MERGED status, and the is_draft
argument is set to None instead of 'true' or 'false' strings so it fails
on
capitalize.
Change-Id: I54b80db84bebb013d82d6f34ed37da7e15b14e48
FIXES: OVIRT-814
Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com>
---
M hooks/custom_hooks/set_modified
M hooks/custom_hooks/update_tracker
2 files changed, 22 insertions(+), 21 deletions(-)
Approvals:
Shlomo Ben David: Verified; Looks good to me, approved
Pavel Zhukov: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/66182
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I54b80db84bebb013d82d6f34ed37da7e15b14e48
Gerrit-PatchSet: 3
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>