Change in gerrit-admin[master]: Include master branch in hook consideration
eedri at redhat.com
eedri at redhat.com
Sun Aug 21 14:26:09 UTC 2016
Eyal Edri has submitted this change and it was merged.
Change subject: Include master branch in hook consideration
......................................................................
Include master branch in hook consideration
until now master branch wasn't in the logic
of deciding if to move bug to MODIFIED or not.
this was because we searched for branch-suffix
matching x.y.z, while master will always have
empty match for that.
this patch should include master in the check
and if there are newer branches that match
the target milestone, the status change
will be skipped
Change-Id: I963d5fd32f8bcf757f8608b60b84225ea87d7c08
Signed-off-by: Eyal Edri <eedri at redhat.com>
Signed-off-by: Shlomi Ben-David <sbendavi at redhat.com>
---
M hooks/custom_hooks/change-merged.set_MODIFIED
1 file changed, 18 insertions(+), 6 deletions(-)
Approvals:
Shlomo Ben David: Verified
Eyal Edri: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/61073
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I963d5fd32f8bcf757f8608b60b84225ea87d7c08
Gerrit-PatchSet: 3
Gerrit-Project: gerrit-admin
Gerrit-Branch: master
Gerrit-Owner: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
More information about the Engine-commits
mailing list