[RFC PATCH 0/2] doc: clarify review replies and reroll timing

Hi,

This small series updates the 2 documentations: MyFirstContribution and SubmittingPatches.

The first patch clarifies that review feedback should not be answered only by sending a new version of the patches, which is talked in [1]. Contributors are encouraged to and should discuss their planned response in the existing review thread, so that the next version does not become the only place where reviewers can infer the author's reasoning.

The second patch is originally from an email from Patrick [2], which documents a rough expectation around reroll frequency.

Patrick suggests: There is no hard rule for when to send a new version, but batching feedback and avoiding multiple rerolls of the same series in a single day is a useful default. The text also mentions factors that may affect this, such as the size of the series, the depth of review, and whether the topic is close to being picked up.

Since I am the newbie here, please tell me how to attribute the credit to Patrick. Thank you Patrick!

I am sending this as RFC because I think my wording is quite rough, and there must be a better way to express all of these, including how to manage the structures of both these two documents. Any comments are appreciated, thank you!

[1]: <xmqq7bo5nf31.fsf@gitster.g> [2]: <aietF4BX1Ewt3cpG@pks.im>

Weijie Yuan (2):
  doc: encourage review replies before rerolling
  doc: advise batching patch rerolls

 Documentation/MyFirstContribution.adoc | 27 +++++++++++++++++++++-----
 Documentation/SubmittingPatches        | 17 +++++++++++++---
 2 files changed, 36 insertions(+), 8 deletions(-)