Re: [PATCH 08/24] t9300 (fast-import), series A: re-indent

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Sep 24, 2010, 07:35 UTC
Message-ID: <20100924073559.GZ4666@burratino>
In-Reply-To: <AANLkTimjD6LqMgQSez=t9N_KPsXaFXm3sMcrZc=qQug+@mail.gmail.com>

Sverre Rabbelier wrote:

> On Fri, Sep 24, 2010 at 09:16, Jonathan Nieder <jrnieder@gmail.com> wrote:

Show 8 quoted lines

>>        test_expect_success \
>>                'series A: foo bar baz' \
>
> Sometimes this style is used if the test description is otherwise too long.
>
>>        test_expect_success 'series A: foo bar baz' '
>
> So what do you do in the case that the line becomes too long?

In this series that never happens.

In general, either

 test_expect_success \
		'series A: foo bar baz' '
	... commands ...
 '

or

 test_expect_success \
	'series A: foo bar baz' '
	... commands ...
 '

looks fine to me for those cases where no shorter test description comes quickly to mind.

$ 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.