Subject: Re: [PATCH v2 3/3] sequencer: comment commit messages properly

From: Junio C Hamano <[email-masked]>
Date: Wed, 13 Nov 2024 10:03:05 +0900

[email-masked] writes:

> From: Kristoffer Haugsbakk <[email-masked]>
>

Describe what happens when a custom comment string is used without
the fixed code in this space.

> Co-authored-by: Phillip Wood <[email-masked]>
> Signed-off-by: Kristoffer Haugsbakk <[email-masked]>
> ---

> +test_expect_success 'conflicting fixup -C after fixup with custom comment string' '
> +	test_config core.commentString COMMENT &&
> +	test_when_finished "test_might_fail git rebase --abort" &&
> +	git checkout --detach A3 &&
> +	test_must_fail env FAKE_LINES="1 fixup 2 fixup_-C 4" git rebase -i A &&
> +	echo resolved >A &&
> +	git add A &&
> +	FAKE_COMMIT_AMEND=edited git rebase --continue &&
> +	test_commit_message HEAD <<-\EOF
> +	A3
> +
> +	edited
> +	EOF
> +'

Doing so would allow readers to imagine more easily how this test
would catch breakages when the code is not fixed (or broken again).

Thanks.