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

Re: [PATCH] [GSoC Patch] Modernize Test Path Checking in Git’s Test Suite

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 18, 2025, 23:53 UTC
Message-ID
<xmqqldt1zyia.fsf@gitster.g>
In-Reply-To
<CAPig+cSXiKOBq_+9+uDTFOLf5F8+KPocwXVX2VzPQAxJHQL99w@mail.gmail.com>
Eric Sunshine <sunshine@sunshineco.com> writes:
Show 6 quoted lines
> Referring to the other thread at [*], perhaps this (avoiding `!` in
> front of test_path_*) is yet another clarification which ought to be
> added to the microproject description in order to lead candidates in a
> more profitable direction.
>
> [*]: https://lore.kernel.org/git/CAPig+cRm+sc+Rk-4SuQ5CrPeZLG2Nzz9B7+6OZxCq7tV5mzmBA@mail.gmail.com/
While mentors do that, I'd prefer to see others extend support for 
	test_path_is_file ! this-should-not-be-file
so that the students have a better tool to work with.
> One other recommendation I would make is to restrict the microproject
> submission to just a single test script (rather than updating twelve
> of them) in order to avoid exhausting the pool for other potential
> candidates.

Yes, you made that point in the other thread, and I agree with it 100%.

Thanks.
Previous: Eric SunshineNext: Sampriyo Guin via GitGitGadget
Message 4 of 5 in “Modernize Test Path Checking in Git’s Test Suite”
  1. [GSoC Patch] Modernize Test Path Checking in Git’s Test SuiteSampriyo Guin via GitGitGadget, Mar 18, 2025
  2. Junio C HamanoMar 18, 2025
  3. Eric SunshineMar 18, 2025
  4. Junio C HamanoMar 18, 2025
  5. [GSoC Patch v2]Modernize Test Path Checking: test -(e|f|d)Sampriyo Guin via GitGitGadget, Mar 19, 2025

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.