Skip to content

Added new mergeability check for requested changes

Phil Hughes requested to merge ph/430728/requestChangesMergeCheck into master

What does this MR do and why?

Adds a new mergeability check that will search all the currently assigned reviewers states checking if they have requested any changes. This change does not include a way for a maintainer to override this check right now.

Note The way this check checks the reviewer state might change in the near future if the reviewers state gets added to the review object instead.

#430728 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Merge request reports