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(a)redhat.com>
Signed-off-by: Shlomi Ben-David <sbendavi(a)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(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>