Volume XXII, number 279Tuesday, October 6, 2026Latest message 48 minutes ago

The Git List

News and archive of git@vger.kernel.org, since April 2005

patchRe: [PATCH] SubmittingPatches: abandoning a series

7 messages between Jul 8, 2026 and Jul 8, 2026, from Michael Montalbo, Junio C Hamano, Weijie Yuan.

Plain Markdown or JSON for tools and agents. Diffs are folded; open one to read it.

Michael MontalboJul 8, 2026, 03:31 UTC on lore
Junio C Hamano <gitster@pobox.com> writes:
> +  A discussion on the list might convince you that your changes are
> +  not such a good idea, in which case you are expected to explicitly
> +  retract the topic, to releave the maintainer from having to worry
> +  about it.
s/releave/relieve/

Also, I might reflow the end: ...in which case you are expected to explicitly retract the topic and relieve the maintainer from having to worry about it.

> ...It is a friendly thing to do to tell the list in such a case...

Again a slight reflow suggestion (ofc my own subjective taste): "...As a courtesy, tell the list in such a case so that..."

Finally, maybe a rule of thumb as to how old a topic a topic should get before a "staleness" update is provided would be helpful, or maybe that is too contextually dependent / would potentially introduce some unwanted scheduling contract?

Overall, I think the explicit guidance is helpful. Of course, an author whose topic is going stale may not be on top of updating its status, stale or otherwise, by definition. However, I think providing this advice up-front will encourage authors to be more proactive about taking action (notify of abandonment / handoff) before a topic gets too old.

Junio C HamanoJul 8, 2026, 04:52 UTC in reply to Michael Montalbo on lore
Michael Montalbo <mmontalbo@gmail.com> writes:
Show 5 quoted lines
> Junio C Hamano <gitster@pobox.com> writes:
>> +  A discussion on the list might convince you that your changes are
>> +  not such a good idea, in which case you are expected to explicitly
>> +  retract the topic, to releave the maintainer from having to worry
>> +  about it.
...
Thanks for improvements.
Show 6 quoted lines
> Finally, maybe a rule of thumb as to how old a topic a topic should get
> before a "staleness" update is provided would be helpful, or maybe that
> is too contextually dependent / would potentially introduce some
> unwanted scheduling contract?
>
> Overall, I think the explicit guidance is helpful.

I've been working on streamlining my workflow to manage the "What's cooking" report, and writing down guidelines with concrete numbers.

 * When the discussion seems to reach rough consensus that the
   latest round looks good for 'next', the topic is immediately
   marked as "Will merge to 'next'?" in my draft copy (note: I do
   not want to spam the list with "What's cooking" too often, but it
   is the document I work from, and it is updated multiple times a
   day).
 * After no negative opinions are seen on a topic in "Will merge to
   'next'?" state for about 36 hours, the topic is marked as "Will
   merge to 'next'".  I will merge such topics during the next
   integration cycle (note: I can only have up to two integration
   cycles per day due to time constraints).
 * Imagine that a topic was last updated more than 4 weeks ago.  If
   there are review comments on the topic that are left unanswered
   by the author for more than a week, and if nothing happens in the
   discussion thread other than inquiry on the current status, the
   topic is marked as "stalled".  I will try to notice and ping a
   stalled topic once or twice, but after that I may discard such a
   topic (which by the way I really hate having to decide to do so).
 * After a topic is merged to 'next', if nothing negative that needs
   fixing is discovered for 7 calendar days, the topic is merged to
   'master'.  I may shorten this depending on how complex the topic
   is.
THere may be more, but these are what I can think of offhand.
Weijie YuanJul 8, 2026, 05:27 UTC in reply to Michael Montalbo on lore
Hi Michael,
By the way, off-topic:

I noticed that several times your replies didn't carry with a "In-Reply-To" header, which made me confused about the threads when surfing on the web interface of lore.kernel.org. So also, I cannot pull down the complete thread with b4. If possible, you may check your email clients, though all email clients suck ;-)

Thanks, Weijie

Michael MontalboJul 8, 2026, 05:41 UTC in reply to Weijie Yuan on lore
On Tue, Jul 7, 2026 at 10:27 PM Weijie Yuan <wy@wyuan.org> wrote:
Show 11 quoted lines
>
> Hi Michael,
>
> By the way, off-topic:
>
> I noticed that several times your replies didn't carry with a
> "In-Reply-To" header, which made me confused about the threads when
> surfing on the web interface of lore.kernel.org. So also, I cannot pull
> down the complete thread with b4. If possible, you may check your email
> clients, though all email clients suck ;-)
>
Argh, sorry about that :(

I think my workflow for replying to threads I am not already a part of is flawed. I usually control-click the reply mailto: link on the message in lore.kernel.org/git. That launches Gmail then I manually craft a message and send from there. I will spend some time investigating my workflow more deeply.

Thank you for the helpful note!
Weijie YuanJul 8, 2026, 05:43 UTC in reply to Junio C Hamano on lore
On Tue, Jul 07, 2026 at 09:52:14PM -0700, Junio C Hamano wrote:
Show 39 quoted lines
> Michael Montalbo <mmontalbo@gmail.com> writes:
> [...]
> > Finally, maybe a rule of thumb as to how old a topic a topic should get
> > before a "staleness" update is provided would be helpful, or maybe that
> > is too contextually dependent / would potentially introduce some
> > unwanted scheduling contract?
> >
> > Overall, I think the explicit guidance is helpful.
> 
> I've been working on streamlining my workflow to manage the "What's
> cooking" report, and writing down guidelines with concrete numbers.
> 
>  * When the discussion seems to reach rough consensus that the
>    latest round looks good for 'next', the topic is immediately
>    marked as "Will merge to 'next'?" in my draft copy (note: I do
>    not want to spam the list with "What's cooking" too often, but it
>    is the document I work from, and it is updated multiple times a
>    day).
> 
>  * After no negative opinions are seen on a topic in "Will merge to
>    'next'?" state for about 36 hours, the topic is marked as "Will
>    merge to 'next'".  I will merge such topics during the next
>    integration cycle (note: I can only have up to two integration
>    cycles per day due to time constraints).
> 
>  * Imagine that a topic was last updated more than 4 weeks ago.  If
>    there are review comments on the topic that are left unanswered
>    by the author for more than a week, and if nothing happens in the
>    discussion thread other than inquiry on the current status, the
>    topic is marked as "stalled".  I will try to notice and ping a
>    stalled topic once or twice, but after that I may discard such a
>    topic (which by the way I really hate having to decide to do so).
> 
>  * After a topic is merged to 'next', if nothing negative that needs
>    fixing is discovered for 7 calendar days, the topic is merged to
>    'master'.  I may shorten this depending on how complex the topic
>    is.
> 
> There may be more, but these are what I can think of offhand.
Integrating the above parts into the document seems like a good idea.
btw, do we need to synchronize MyFirstContribution simultaneously?
Quoting Patrick's words [1]:

Overall it's a bit on the annoying side that we have to always make sure to update both SubmittingPatches and MyFirstContribution in tandem. Makes me wonder whether they are mostly redundant and whether it would make sense to eventually merge them.

Thanks, Weijie

[1] https://lore.kernel.org/git/ai_7Wh7hrD8PZozg@pks.im/
Weijie YuanJul 8, 2026, 06:21 UTC in reply to Michael Montalbo on lore
On Tue, Jul 07, 2026 at 10:41:53PM -0700, Michael Montalbo wrote:
Show 5 quoted lines
> I think my workflow for replying to threads I am not already a part of is
> flawed. I usually control-click the reply mailto: link on the message in
> lore.kernel.org/git. That launches Gmail then I manually craft a message
> and send from there. I will spend some time investigating my workflow
> more deeply.

I guess you are right. I suspect the Gmail web interface is the reason why the In-Reply-To header was lost here.

> Thank you for the helpful note!
You´re welcome!
Junio C HamanoJul 8, 2026, 16:33 UTC in reply to Weijie Yuan on lore
Weijie Yuan <wy@wyuan.org> writes:
Show 42 quoted lines
> On Tue, Jul 07, 2026 at 09:52:14PM -0700, Junio C Hamano wrote:
>> Michael Montalbo <mmontalbo@gmail.com> writes:
>> [...]
>> > Finally, maybe a rule of thumb as to how old a topic a topic should get
>> > before a "staleness" update is provided would be helpful, or maybe that
>> > is too contextually dependent / would potentially introduce some
>> > unwanted scheduling contract?
>> >
>> > Overall, I think the explicit guidance is helpful.
>> 
>> I've been working on streamlining my workflow to manage the "What's
>> cooking" report, and writing down guidelines with concrete numbers.
>> 
>>  * When the discussion seems to reach rough consensus that the
>>    latest round looks good for 'next', the topic is immediately
>>    marked as "Will merge to 'next'?" in my draft copy (note: I do
>>    not want to spam the list with "What's cooking" too often, but it
>>    is the document I work from, and it is updated multiple times a
>>    day).
>> 
>>  * After no negative opinions are seen on a topic in "Will merge to
>>    'next'?" state for about 36 hours, the topic is marked as "Will
>>    merge to 'next'".  I will merge such topics during the next
>>    integration cycle (note: I can only have up to two integration
>>    cycles per day due to time constraints).
>> 
>>  * Imagine that a topic was last updated more than 4 weeks ago.  If
>>    there are review comments on the topic that are left unanswered
>>    by the author for more than a week, and if nothing happens in the
>>    discussion thread other than inquiry on the current status, the
>>    topic is marked as "stalled".  I will try to notice and ping a
>>    stalled topic once or twice, but after that I may discard such a
>>    topic (which by the way I really hate having to decide to do so).
>> 
>>  * After a topic is merged to 'next', if nothing negative that needs
>>    fixing is discovered for 7 calendar days, the topic is merged to
>>    'master'.  I may shorten this depending on how complex the topic
>>    is.
>> 
>> There may be more, but these are what I can think of offhand.
>
> Integrating the above parts into the document seems like a good idea.

It could be, but not to the document we are discussing. The current document is about contributor's guide, telling them what to do and how, and "'next' usually holds a topic for 7 days" is not as interesting.

Show 8 quoted lines
> btw, do we need to synchronize MyFirstContribution simultaneously?
>
> Quoting Patrick's words [1]:
>
> Overall it's a bit on the annoying side that we have to always make sure
> to update both SubmittingPatches and MyFirstContribution in tandem.
> Makes me wonder whether they are mostly redundant and whether it would
> make sense to eventually merge them.
Surely, but that is outside the scope of this patch.
Thanks.

Back to recent threads