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

Ghanshyam Thakkar [off-list ref] 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?