Skip to content

Commit 3b54f67

Browse files
bertyuangitster
authored andcommitted
MyFirstContribution: mention trimming quoted text in replies
ReviewingGuidelines already advises reviewers to trim irrelevant quoted context when replying. Give the same advice to new contributors in MyFirstContribution, so our documentation is consistent about mailing list reply etiquette. Signed-off-by: Weijie Yuan <wy@wyuan.org> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent 3e65291 commit 3b54f67

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

Documentation/MyFirstContribution.adoc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1367,6 +1367,11 @@ effect which had not occurred to you. It is always okay to ask for clarification
13671367
if you aren't sure why a change was suggested, or what the reviewer is asking
13681368
you to do.
13691369

1370+
When replying to review comments, quote only the parts of the message that are
1371+
relevant to your response. It is usually helpful to trim away unrelated context,
1372+
such as large portions of the patch that are not being discussed, while keeping
1373+
enough quoted text for readers to understand what you are responding to.
1374+
13701375
Make sure your email client has a plaintext email mode and it is turned on; the
13711376
Git list rejects HTML email. Please also follow the mailing list etiquette
13721377
outlined in the

0 commit comments

Comments
 (0)