Subject: Re: [PATCH v2 1/2] t7501: add tests for --include, --only of commit

From: Junio C Hamano <[email-masked]>
Date: Tue, 09 Jan 2024 09:50:04 -0800

Ghanshyam Thakkar <[email-masked]> writes:

> This commit adds tests for testing --include (-o) and --only (-o). It

"-i" not "-o" stands for "--include".

Please write in imperative mood.

> +test_expect_success '--include fails with untracked files' '
> +	echo content >baz &&
> +	test_must_fail git commit --include baz -m initial
> +'

My comment on argument order applies to this iteration, too.  Please
write your code to help readers.

> +test_expect_success 'commit --include' '
> +	test_when_finished "rm -rf remaining" &&

Why recursive removal when you _know_ what you create is a plan
file?