From 09f1f51e9e4289906ea447dbdefb1ca27cf8764e Mon Sep 17 00:00:00 2001 From: Basil Crow Date: Fri, 17 Dec 2021 07:56:53 -0800 Subject: [PATCH] Only require one approval for minor changes (#6051) Co-authored-by: Daniel Beck <1831569+daniel-beck@users.noreply.github.com> Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com> --- docs/MAINTAINERS.adoc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/MAINTAINERS.adoc b/docs/MAINTAINERS.adoc index 3218936cc6..f8fcf38314 100644 --- a/docs/MAINTAINERS.adoc +++ b/docs/MAINTAINERS.adoc @@ -374,6 +374,9 @@ LTS backporting, if needed, will be handled separately by the release team. Security hardening and enhancements go through the standard process. * Release Team members are permitted to bypass the review/merge process if and only if a change is needed to unblock the security release. Common review process is used otherwise. +* Only one approval is required for low-risk small changes with the `skip-changelog` label, + as long as both author and approver have write access to the repository. + Note that the 24-hour waiting period still applies. * 24 hours waiting period after adding the `ready-for-merge` label is not required for: //TODO(oleg_nenashev): Define "trivial" better to avoid loopholes ** changes that do not result in changes to the primary functionality, such as typo fixes in documentation or localization files