diff --git a/dr_botzo/gitlab_bot/lib.py b/dr_botzo/gitlab_bot/lib.py index 8b731e8..e7316a4 100644 --- a/dr_botzo/gitlab_bot/lib.py +++ b/dr_botzo/gitlab_bot/lib.py @@ -127,8 +127,11 @@ class GitlabBot(object): log.debug("merge request '%s', note '%s' is a system message", merge_request.title, note.id) if re.match(r'Added \d+ commit', note.body): log.debug("resetting approval list, '%s' looks like a push!", note.body) + + # only set the unlogged approval reset flag if there's some kind of progress + if len(request_state.approver_list) > 0: + request_state.unlogged_approval_reset = True request_state.approver_list.clear() - request_state.unlogged_approval_reset = True else: log.debug("leaving the approval list as it is, i don't think '%s' is a push", note.body)