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

Re: [PATCH] area: /t/t4204-log.sh, partially modernized test script t4202

From
Junio C Hamano <gitster@pobox.com>
Date
Apr 20, 2022, 06:13 UTC
Message-ID
<xmqqmtggs2nv.fsf@gitster.g>
In-Reply-To
<pull.1218.git.1650096550631.gitgitgadget@gmail.com>
"Jack McGuinness via GitGitGadget" <gitgitgadget@gmail.com> writes:
> From: Jack <jmcguinness2@ucmerced.edu>
> Subject: Re: [PATCH] area: /t/t4204-log.sh, partially modernized test script t4202

I am not sure what "area" in this context is. Do not report what you did in past tense on the title.

Perhaps
    Subject: [PATCH] t4202: modernize code layout
or something?
> Remove test body indentations made with spaces, replace with tabs.
> Remove blank lines at start and end of test bodies.

OK. The title says "partially", strongly hinting that the script is not fully modern even if we take this patch. There should be some mention of left-over work that is left to a later occasion here.

Thanks.
Previous: Derrick Stolee
Message 3 of 3 in “area: /t/t4204-log.sh, partially modernized test script t4202”
  1. area: /t/t4204-log.sh, partially modernized test script t4202Jack McGuinness via GitGitGadget, Apr 16, 2022
  2. Derrick StoleeApr 18, 2022
  3. Junio C HamanoApr 20, 2022

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.