Git development
 help / color / mirror / Atom feed
From: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>
To: Tatchou Yopa Nelly <yopatatchou@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [Outreachy][PATCH] promisor-remote: honor promisor.fetchOrder when trying remotes
Date: Fri, 13 Nov 2026 15:21:09 +0530	[thread overview]
Message-ID: <a3ae6756-0985-4873-9d23-96d4bb49eda5@gmail.com> (raw)
In-Reply-To: <CAG7jffxRscX3ntX3p8L1GwGbs7WJmj6WoKPRGUuz3wcCS0NZag@mail.gmail.com>

On 11/6/26 04:25, Tatchou Yopa Nelly wrote:
> Hi Tanga,
>

Next time try to make sure the person you are writing to is in the To/Cc 
fields as appropriate. Otherwise, there is a chance that they will not 
even receive your message since not everyone reads the list by 
subscribing to it.> Kaartic, I've also re-read 
SubmittingPatches#describe-changes and
> rewritten the commit message to describe the change rather than
> address the reviewers directly.

That is good!

> On the RFC point: I treated the
> Phase 2 config work as the micro-patch and skipped the RFC step
> you pointed out. Should I step back and send the design RFC first,
> or keep the current patch as a prototype while we discuss the
> config shape on the list?
> 

I think treating the current patch as a RFC patch and discussing the 
shape is a good idea. The only concern I have is that this patch is 
submitted in the context of Outreachy and the candidate that we may 
potentially pick will work[note 1] on this during their Outreachy 
internship period. So, discussing this now sounds a little premature to me

Of course, I don't mean to suggest that no two people should work on 
related areas. I'm just thinking it might be ideal to avoid premature 
work among Outreachy contributors before even the Outreachy selections 
are out.


[note 1]: During the candidate discussion, it may also happen that we 
may not pick a candidate for the project. It is totally fine to work on 
this in that case as there will be no conflicts. But none of this is 
clear now, TBH.

-- 
Sivaraam


  reply	other threads:[~2026-11-13  9:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-04 17:24 [Outreachy][PATCH] promisor-remote: honor promisor.fetchOrder when trying remotes Yopa Nelly
2026-11-05 14:30 ` Tanga Joris
2026-11-05 16:31 ` Kaartic Sivaraam
2026-11-05 22:55   ` Tatchou Yopa Nelly
2026-11-13  9:51     ` Kaartic Sivaraam [this message]
2026-11-13 12:13       ` Tatchou Yopa Nelly
2026-11-05 22:52 ` [Outreachy][PATCH v2] " Yopa Nelly
2026-11-05 23:09   ` Junio C Hamano
2026-11-13 17:10     ` Christian Couder
2026-11-13 19:46       ` Tatchou Yopa Nelly
2026-11-13 19:58       ` Junio C Hamano

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=a3ae6756-0985-4873-9d23-96d4bb49eda5@gmail.com \
    --to=kaartic.sivaraam@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=yopatatchou@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
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