On Fri, May 27, 2022 at 03:36:12PM +0200, Peter Krempa wrote:
Add a note outling best practices around review and responding to
it.
Signed-off-by: Peter Krempa <pkrempa(a)redhat.com>
---
docs/submitting-patches.rst | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/docs/submitting-patches.rst b/docs/submitting-patches.rst
index 7cb5c2e172..d6bc9b623c 100644
--- a/docs/submitting-patches.rst
+++ b/docs/submitting-patches.rst
@@ -89,3 +89,18 @@ Moreover, such patch needs to be prefixed correctly with
``--subject-prefix=PATCHv2`` appended to
``git send-email`` (substitute ``v2`` with the
correct version if needed though).
+
+Reviews
+-------
+
+Please note that reviewing may take a lot of effort so be patient. Try to react
Nit: I'd start the sentences with "Note" and end it with "so please be
patient"
"Trying" to respond to questions may not result in the desired outcome, so
I'd
make it more explicit - "Respond to reviewer's questions"
+to questions from reviewers. There's no need to acknowledge
individual points
+before addressing them in next version. Make sure to address all of reviewers
+requests, or start a discussion if you don't agree with reviewers points.
+
+Feel free to respond to any flaw in any patch on the list, do not feel obliged
+to review the rest of the series in such case.
^This paragraph is a note for the reviewers...
+
+If your patches don't get attention after some time (~week) feel free to
+remind us on list that your patches still need review. **Do not** contact
+individual maintainers directly to review your patches.
...while this one is again a note for contributors, IOW they should not be
split.
Overall I like the idea, but I think it would be better if we formatted this as
a list of "Dos and Don'ts" (In fact we can even name the section this way).
You'd end up with something like:
- **do** be patient with reviewers, reviews take a lot of effort and hence may
take some time
- **do** respond to reviewer's questions
- **don't** contact individual maintainers/developers directly with your
patches
- **do** remind us of your patches on the list if they haven't gotten any
attention for a prolonged period (> week) by replying to your patches with a
"ping"
Notes for reviewers:
- **don't** feel obliged to do a full patch review if you spot a serious flaw
anywhere in the series
Take ^^^this just as an alternative idea of the same concept.
Erik