git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] t0081: modernize the code testing whether a file exists

From
Junio C Hamano <gitster@pobox.com>
Date
Oct 15, 2026, 17:10 UTC
Message-ID
<xmqqzeweopov.fsf@gitster.g>
In-Reply-To
<9c905a79-22a2-4db2-b434-00ad8a287300@gmail.com>
Kaartic Sivaraam <kaartic.sivaraam@gmail.com> writes:
Show 18 quoted lines
> That said, I just wanted to highlight one thing. In the Microproject 
> ideas page[1], we have the following for this specific idea:
>
>    Notes
>
>    - Start small: Pick a test file with just a few instances to convert
>    - Run the test suite after your changes to ensure nothing breaks
>    - Follow Git’s commit message style
>    - Include which command you used to find the instances in your commit
>      message
>
> So, we can't really blame the candidates for including the command in 
> the commit message. It is our mistake asking them to include it in the 
> first place when we don't prefer it to be present there.
>
> I raised a PR to fix that:
>
>    https://github.com/git/git.github.io/pull/875

I do not think your update is taking us in the right direction, though. Recommend to have it under three-dash lines instead, perhaps?

Previous: Kaartic SivaraamNext: Kaartic Sivaraam
Message 6 of 9 in “t0081: modernize the code testing whether a file exists”
  1. t0081: modernize the code testing whether a file existsShashank S via GitGitGadget, Oct 14, 2026
  2. Usman AkinyemiOct 14, 2026
  3. Junio C HamanoOct 14, 2026
  4. Shashank SOct 15, 2026
  5. Kaartic SivaraamOct 15, 2026
  6. Junio C HamanoOct 15, 2026
  7. Kaartic SivaraamOct 23, 2026
  8. Shashank SOct 15, 2026
  9. [Outreachy] t0081: modernize the code testing whether a file existsShashank S via GitGitGadget, Oct 15, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.