Re: [PATCH] t0081: modernize the code testing whether a file exists
On 10/15/26 22:40, Junio C Hamano wrote:
Show 7 quoted lines
> Kaartic Sivaraam <kaartic.sivaraam@gmail.com> 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
Kaartic Sivaraam <kaartic.sivaraam@gmail.com>