On Fri, May 16, 2025 at 12:11 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > JAYATHEERTH K <jayatheerthkulkarni2005@xxxxxxxxx> writes: > > > On Fri, May 16, 2025 at 4:09 AM Emily Shaffer <nasamuffin@xxxxxxxxxx> wrote: > > > >> Mostly I lurk these days :) I do still keep an eye on the list. Will > >> happily take a look at your series tomorrow, I'm out of time for > >> today. But per what I mention below, if you don't hear from me, please > >> don't feel blocked by the review, as I think the MyFirstContribution > >> doc is comfortably maintained by the whole project by now. > >> > > > > Understood!! thanks for letting me know > > > >> > So for now I will cc Philippe > >> > >> For what it's worth, I don't think it is harmful to CC people even if > >> they will be inactive. CCing someone is not necessarily the same thing > >> as saying that person needs to approve your code change, right? So I > >> don't see the harm in CCing with low expectations - in fact, in my > >> case it would help make the email stand out, so you'd be more likely > >> to get a review from me (I missed this thread going by initially). > >> > >> > > > > Oh, ok I will keep that in mind next time. > > > >> - Emily > > > > Thank you, > > Thanks for a pleasant conversation; others can also learn from this > exchange, hopefully. In Documentation/SubmittingPatches we have > "Choosing your reviewers" section lacks anything more concrete than > "who are involved in the area you are touching", and those who use > common sense may say, just like you did, "ah, most of the text I am > replacing was written N years ago by person X, whom I no longer see > on the list very often" and decide to omit it. Perhaps we would > want to enhance the text there somewhat? I dunno. > > Since there were discussions on contrib/contacts recently (a few of > the participants there added to CC), I tried it and unfortunately I > was not very impressed by its output [*]. > > After applying the four patches on top of 'master', you'd run the > tool like so: > > $ contrib/contacts/git-contacts master.. > Jonathan Nieder <jrnieder@xxxxxxxxx> > Jacob Stopak <jacob@xxxxxxxxxxxxxxxx> > Jeff King <peff@xxxxxxxx> > Jean-Noël Avila <jn.avila@xxxxxxx> > Emily Shaffer <nasamuffin@xxxxxxxxxx> > Atharva Raykar <raykar.ath@xxxxxxxxx> > Junio C Hamano <gitster@xxxxxxxxx> > Todd Zullinger <tmz@xxxxxxxxx> > Kyle Lippincott <spectral@xxxxxxxxxx> > > The tool gave output in a different order every time it was run. It > wasn't obvious what the ordering meant. > > By looking at its source, I can tell that the names and addresses > are collected from trailers like reported-by, which are counted with > the same importance as the authorship, that the reason why the > output is different each time it is run is due to use of keys %hash > in a Perl script, etc., but counting sign-off would mean that I'd be > summoned for each and every change related in this project, which > would not be very productive use of everybody's time. > > And it of course is not clear who are still active in the recent > past and why the name was in the list (it would not be as productive > to ask for a review from somebody who was listed for reporting many > problems in the area affected by the proposed patch than those who > wrote the original) from this output. There may want an "explain" > mode that lets you feed a patch and get observations like: > > The majority of lines you are touching haven't changed much > since person X wrote commit W 5 years ago, and the text turned > into current shape with contributions by person Y and Z. Here > are the URLs into the lore archive for the discussion that you > can see how X, Y, and Z participated in the original before you > touched. You may also want to look at commit V and U as well. > > Last time we saw person X, Y, and Z on the list were ..., here > are the URLs into the lore archive. > > Perhaps some AI minded folks can write such a service for us ;-)? > > > [Footnote] > > * I didn't try other alternatives which I didn't have, and the > other thread there was a mention of "git related" with "seems > like rather more work". > > cf. https://lore.kernel.org/git/aBr9bwNQ1J46NNXI@xxxxxx/ > I sometimes use another alternative "git overwritten" [1] which counts blame information instead. I've not tried to modify it to be compatible with contacts, though. For example, after applying the initial version of patches (which had whitespace errors, btw; didn't check the latest version) on 7a1d2bd0a5 (Merge branch 'master' of https://github.com/j6t/gitk, 2025-05-09), "git overwritten" shows 10 76644e3268 (documentation: add tutorial for first contribution, 2019-05-17) by Emily Shaffer 5 a2dc43414c (MyFirstContribution: rephrase contact info, 2020-02-13) by Emily Shaffer 3 4bb4fd4290 (MyFirstContribution: add avenues for getting help, 2020-01-24) by Emily Shaffer 2 9a53219f69 (config: drop git_config_get_string_const(), 2020-08-17) by Jeff King 2 2656fb16dd (doc: add some nit fixes to MyFirstContribution, 2019-05-29) by Emily Shaffer 1 8b4b41aefb (MyFirstContribution: *.txt -> *.adoc fixes, 2025-03-03) by Todd Zullinger [1]: https://github.com/benknoble/Dotfiles/blob/master/links/bin/git-overwritten -- D. Ben Knoble
