From: "brian m. carlson" <sandals@crustytoothpaste•net>
To: Justin Tobler <jltobler@gmail•com>
Cc: git@vger•kernel.org, ps@pks•im
Subject: Re: [PATCH] Documentation: extend guidance for submitting patches
Date: Fri, 6 Mar 2026 12:58:54 +0000 [thread overview]
Message-ID: <aarPjlYv0H10rNKJ@fruit.crustytoothpaste.net> (raw)
In-Reply-To: <20260305193836.973122-1-jltobler@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1589 bytes --]
On 2026-03-05 at 19:38:36, Justin Tobler wrote:
> +It is also a good idea to check whether your topic has been discussed
> +previously on the mailing list, or whether similar work is already in
> +progress. Prior discussions may contain useful context, design
> +considerations, or earlier attempts at solving the same problem. Being
> +aware of such discussions can help you avoid duplicating work and may
> +allow you to coordinate with other contributors working in the same
> +area.
This seems reasonable. We've had cases of patch series that have
stalled due to a minor issue and someone wanting to send a patch may
find that they really could fix that minor issue on top of the existing
patch and have their problem solved. Or at least, they might be
inclined to not get stuck in the same way.
I try to do this anyway, but it's much easier on forge-style systems
than it is on a mailing list, so mentioning it may help refresh people's
memories.
You could, if you wanted to, link to `{gitml}` after the phrase “the
mailing list,” which might help folks find the right location. Or you
could link to the https://lore.kernel.org/git/ archives instead via a
footnote.
> +It is often beneficial to allow some time for reviewers to provide
> +feedback before sending a new version, rather than sending an updated
> +series immediately after receiving a review. This helps collect broader
> +input and avoids unnecessary churn from many rapid iterations.
I think this is a good idea, too.
--
brian m. carlson (they/them)
Toronto, Ontario, CA
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 262 bytes --]
prev parent reply other threads:[~2026-03-06 12:58 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-05 19:38 [PATCH] Documentation: extend guidance for submitting patches Justin Tobler
2026-03-05 20:35 ` Junio C Hamano
2026-03-05 21:27 ` Justin Tobler
2026-03-05 21:35 ` Junio C Hamano
2026-03-05 21:43 ` Justin Tobler
2026-03-05 22:27 ` Kristoffer Haugsbakk
2026-03-06 1:48 ` Justin Tobler
2026-03-06 10:56 ` Kristoffer Haugsbakk
2026-03-06 12:58 ` brian m. carlson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=aarPjlYv0H10rNKJ@fruit.crustytoothpaste.net \
--to=sandals@crustytoothpaste$(echo .)net \
--cc=git@vger$(echo .)kernel.org \
--cc=jltobler@gmail$(echo .)com \
--cc=ps@pks$(echo .)im \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox