{"thread":{"id":"65790","subject":"[RFC PATCH] MyFirstContribution: mention trimming quoted text in replies","startedAt":"2026-06-11T14:15:08Z","lastAt":"2026-06-12T08:39:44Z","messageCount":3,"participants":["Weijie Yuan","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"545284","messageId":"080402ff0ac8127b654dccea59a1bf643df62a5c.1781186476.git.wy@wyuan.org","threadId":"65790","inReplyTo":null,"subject":"[RFC PATCH] MyFirstContribution: mention trimming quoted text in replies","fromName":"Weijie Yuan","fromEmail":"wy@wyuan.org","sentAt":"2026-06-11T14:14:35Z","receivedAt":"2026-06-11T14:15:08Z","isPatch":true,"body":"Hi,\n\nJunio has pointed out a few times that over-quoting can make review\ndiscussions harder to follow [1], so I thought it would be helpful to\nadd a reminder about it in MyFirstContribution. Please feel free to\ncomment on the patch, as my English is not good.\n\nFor your reference:\n\nGit's ReviewingGuidelines [2]:\n\nIf a patch is long, you are encouraged to delete parts of it that are\nunrelated to your review from the email reply. Make sure to leave enough\ncontext for readers to understand your comments!\n\nThe Linux Kernel documentation [3]:\n\nSimilarly, please trim all unneeded quotations that aren´t relevant to\nyour reply. This makes responses easier to find, and saves time and\nspace.\n\n[1] \n[2] https://git-scm.com/docs/ReviewingGuidelines/2.38.0#_performing_your_review\n[3] https://docs.kernel.org/process/submitting-patches.html#use-trimmed-interleaved-replies-in-email-discussions\n\n--- >8 ---\n\nReviewingGuidelines already advises reviewers to trim irrelevant quoted\ncontext when replying. Give the same advice to new contributors in\nMyFirstContribution, so our documentation is consistent about mailing\nlist reply etiquette.\n\nSigned-off-by: Weijie Yuan \n---\n Documentation/MyFirstContribution.adoc | 5 +++++\n 1 file changed, 5 insertions(+)\n\ndiff --git a/Documentation/MyFirstContribution.adoc b/Documentation/MyFirstContribution.adoc\nindex 607876f3d8..0e2a9313ce 100644\n--- a/Documentation/MyFirstContribution.adoc\n+++ b/Documentation/MyFirstContribution.adoc\n@@ -1453,6 +1453,11 @@ effect which had not occurred to you. It is always okay to ask for clarification\n if you aren't sure why a change was suggested, or what the reviewer is asking\n you to do.\n \n+When replying to review comments, quote only the parts of the message that are\n+relevant to your response. It is usually helpful to trim away unrelated context,\n+such as large portions of the patch that are not being discussed, while keeping\n+enough quoted text for readers to understand what you are responding to.\n+\n Make sure your email client has a plaintext email mode and it is turned on; the\n Git list rejects HTML email. Please also follow the mailing list etiquette\n outlined in the\n-- \n2.54.0\n\n"},{"id":"545319","messageId":"xmqqcxxwljue.fsf@gitster.g","threadId":"65790","inReplyTo":"080402ff0ac8127b654dccea59a1bf643df62a5c.1781186476.git.wy@wyuan.org","subject":"Re: [RFC PATCH] MyFirstContribution: mention trimming quoted text in replies","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-06-11T23:48:25Z","receivedAt":"2026-06-11T23:48:27Z","isPatch":true,"body":"Weijie Yuan writes:\n\n> ReviewingGuidelines already advises reviewers to trim irrelevant quoted\n> context when replying. Give the same advice to new contributors in\n> MyFirstContribution, so our documentation is consistent about mailing\n> list reply etiquette.\n>\n> Signed-off-by: Weijie Yuan \n\nMakes sense.\n\n> diff --git a/Documentation/MyFirstContribution.adoc b/Documentation/MyFirstContribution.adoc\n> index 607876f3d8..0e2a9313ce 100644\n> --- a/Documentation/MyFirstContribution.adoc\n> +++ b/Documentation/MyFirstContribution.adoc\n> @@ -1453,6 +1453,11 @@ effect which had not occurred to you. It is always okay to ask for clarification\n> if you aren't sure why a change was suggested, or what the reviewer is asking\n> you to do.\n> \n> +When replying to review comments, quote only the parts of the message that are\n> +relevant to your response. It is usually helpful to trim away unrelated context,\n> +such as large portions of the patch that are not being discussed, while keeping\n> +enough quoted text for readers to understand what you are responding to.\n> +\n> Make sure your email client has a plaintext email mode and it is turned on; the\n> Git list rejects HTML email. Please also follow the mailing list etiquette\n> outlined in the\n\nThe insertion point is well chosen, immediately following the\ndiscussion on how to handle review comments and before the technical\ndetails of email client configuration. The text itself is clear and\ngives sound advice.\n\nWill queue and wait for others to weigh in. Thanks.\n"},{"id":"545353","messageId":"aivFyBanzoHnYRsy@wyuan.org","threadId":"65790","inReplyTo":"xmqqcxxwljue.fsf@gitster.g","subject":"Re: [RFC PATCH] MyFirstContribution: mention trimming quoted text in replies","fromName":"Weijie Yuan","fromEmail":"wy@wyuan.org","sentAt":"2026-06-12T08:39:36Z","receivedAt":"2026-06-12T08:39:44Z","isPatch":true,"body":"On Thu, Jun 11, 2026 at 04:48:25PM -0700, Junio C Hamano wrote:\n> [...]\n> > diff --git a/Documentation/MyFirstContribution.adoc b/Documentation/MyFirstContribution.adoc\n> > index 607876f3d8..0e2a9313ce 100644\n> > --- a/Documentation/MyFirstContribution.adoc\n> > +++ b/Documentation/MyFirstContribution.adoc\n> > @@ -1453,6 +1453,11 @@ effect which had not occurred to you. It is always okay to ask for clarification\n> > if you aren't sure why a change was suggested, or what the reviewer is asking\n> > you to do.\n> > \n> > +When replying to review comments, quote only the parts of the message that are\n> > +relevant to your response. It is usually helpful to trim away unrelated context,\n> > +such as large portions of the patch that are not being discussed, while keeping\n> > +enough quoted text for readers to understand what you are responding to.\n> > +\n> > Make sure your email client has a plaintext email mode and it is turned on; the\n> > Git list rejects HTML email. Please also follow the mailing list etiquette\n> > outlined in the\n> \n> The insertion point is well chosen, immediately following the\n> discussion on how to handle review comments and before the technical\n> details of email client configuration. The text itself is clear and\n> gives sound advice.\n\nThank you for your reply.\n\nI first considered folding this into one of the existing paragraphs\nabove, but ended up making it a separate paragraph for better clarity.\nLet's see whether others have better wording suggestions.\n\nNow the expectations for contributors and reviewers are more aligned\nwhen participating in review discussions :-)\n\nHopefully this addition will make review discussions a little easier to\nfocus and reduce the burden on reviewers.\n\nThanks!\n"}]}