Thread (15 messages) flat view 15 messages, 2 authors, 2016-06-15

Re: [PATCHv3 1/6] Add additional testcases for D/F conflicts

From: Elijah Newren <hidden>
Date: 2016-06-15 22:49:06

Hi,

Thanks for the thorough reviews!

On Wed, Jul 7, 2010 at 3:25 PM, Junio C Hamano [off-list ref] wrote:
quoted
-test_expect_failure 'do not lose a/b-2/c/d in merge (recursive)' '
+test_expect_failure 'Handle D/F conflict, do not lose a/b-2/c/d in merge (recursive)' '
If you retitle this, you would probably want to retitle the corresponding
test for resolve (the one before this test) the same way.
<snip>
It may be a good idea to see what resolve does for all these new
tests.
I'd be happy to make these and the other changes you suggested, but I
notice that you've already done so in pu with Fixup commits for this
and the other patches.  Should I squash those in for a resend of the
series?

Also, do you want me to split the series as you did with the testsuite
addition patches separate from the fixes?

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