From e36fccbbf9348deeaf682b3b968e43a27867ff64 Mon Sep 17 00:00:00 2001 From: Aleksandar Markovic Date: Thu, 21 Apr 2022 18:29:33 +0200 Subject: [PATCH] doc: Reword and reformat a sentence on review time Move the sentence to the end of the section on labels applicable to PRs only, reword it and convert to a note, indicating that the note is for all labels in this section. Signed-off-by: Aleksandar Markovic --- doc/project/dev_env_and_tools.rst | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/doc/project/dev_env_and_tools.rst b/doc/project/dev_env_and_tools.rst index 6f99e6f01ec..d3032487db7 100644 --- a/doc/project/dev_env_and_tools.rst +++ b/doc/project/dev_env_and_tools.rst @@ -328,9 +328,6 @@ Maintainer review reqiured. To be reviewed by a security expert. -Depending on the PR complexity, an indication of how long a merge should be held -to ensure proper review. See :ref:`review process `. - * *DNM* This PR should not be merged (Do Not Merge). For work in progress, GitHub @@ -348,6 +345,12 @@ The PR is a backport or should be backported. The PR has licensing issues which require a licensing expert to review it. +.. note:: + For all labels applicable to PRs: Please note that the label, together with + PR complexity, affects how long a merge should be held to ensure proper + review. See :ref:`review process ` for details. + + Labels applicable to both pull requests and issues ==================================================