Ghanshyam Thakkar <shyamthakkar001@xxxxxxxxx> 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?
