Stefan Beller <sbel...@google.com> writes: > Signed-off-by: Stefan Beller <sbel...@google.com> > --- > t/t6120-describe.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
Good. I am guessing that you are sending this as the last/optional one because this was found _after_ you worked on other parts of the series, but I think it is easier to reason about if this were marked as a preliminary clean-up and moved to the front of the series. Thanks. > > diff --git a/t/t6120-describe.sh b/t/t6120-describe.sh > index 3be01316e8..fd329f173a 100755 > --- a/t/t6120-describe.sh > +++ b/t/t6120-describe.sh > @@ -304,7 +304,7 @@ test_expect_success 'describe chokes on severely broken > submodules' ' > mv .git/modules/sub1/ .git/modules/sub_moved && > test_must_fail git describe --dirty > ' > -test_expect_success 'describe ignoring a borken submodule' ' > +test_expect_success 'describe ignoring a broken submodule' ' > git describe --broken >out && > test_when_finished "mv .git/modules/sub_moved .git/modules/sub1" && > grep broken out
