Skip to content

Fix required approvers validation

Marcos Rocha requested to merge mc_rocha-fix-required-approvals-validation into master

What does this MR do and why?

This MR updates the required_approvals_exceed_eligible_approvers? check to consider inherited members for group approvals.

More context in this thread(internal-only).

Database

https://console.postgres.ai/gitlab/gitlab-production-main/sessions/29624/commands/91944

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.

Edited by Dominic Bauer

Merge request reports