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>
This commit is contained in:
authored and GitHub committed 2021-12-17 07:56:53 -08:00
1 parent b0c61ecac1
commit 09f1f51e9e
1 file changed
+3
+3
View File
@@ -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