From: Kaartic Sivaraam Date: Fri, 23 Oct 2026 06:56:51 GMT Subject: Re: [PATCH] t0081: modernize the code testing whether a file exists Message-ID: <2a050885-a1c2-48d6-b561-3c2ea0a05dbf@gmail.com> In-Reply-To: On 10/15/26 22:40, Junio C Hamano wrote: > Kaartic Sivaraam writes: > >> 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? Got it. Does the following change read better? diff --git a/links/mentoring/soc/SoC-2026-Microprojects.md b/links/mentoring/soc/SoC-2026-Microprojects.md index 41354b50..17519e2d 100644 --- a/links/mentoring/soc/SoC-2026-Microprojects.md +++ b/links/mentoring/soc/SoC-2026-Microprojects.md @@ -49,8 +49,9 @@ with Git's dedicated test helpers like `test_path_is_file`. - 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 -- You _do not need to_ include the command you used to find the instances - in your commit message +- You can include the command you used to find the instances in your + cover letter or between the three-dash line and the diffstat, whichever + is appropriate. Do not mention it in your commit message #### Need Help? - Reference [this discussion](https://public-inbox.org/git/CAPig+cRfO8t1tdCL6MB4b9XopF3HkZ==hU83AFZ38b-2zsXDjQ@mail.gmail.com/) -- Sivaraam