Thread (1 message) 1 message, 1 author, 2021-09-07

Re: [PATCH v2] apply: resolve trivial merge without hitting ll-merge with "--3way"

From: Junio C Hamano <hidden>
Date: 2021-09-07 20:15:40

Ævar Arnfjörð Bjarmason [off-list ref] writes:
Partly this is cleaning up a mess after an existing test, but here
there's no reason we can't use test_when_finished() for all the new
tests to make them clean up after themselves:
I do not mind if somebody wants to send in a janitorial patch after
the dust settles, but adding "test_when_finished reset --hard" after
each "refs --hard" at the beginning of each test is not something I
would expect to see.  Such a patch should first choose between "each
test cleans after itself" and "expect previous ones may have left a
mess, so each test clears the slate sufficiently before it starts"
and then stick to the approach, not mixture of both, I would think.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help