[darcs-devel] [patch1915] Add a test for rebasing a pre-existing c... (and 1 more)

Ben Franksen bugs at darcs.net
Sun Sep 22 07:29:21 UTC 2019


Ben Franksen <ben.franksen at online.de> added the comment:

>> I think the point here is not how complicated the conflict or scenario
>> is. The question is whether we eliminate any patches that the conflicted
>> patch conflicts with, because that invalidates (at least for V3)
>> Conflictors.
> 
> I guess "simple" is a bit ambiguous and I could have just said
> "some scenarios".
> 
> You set this to review-in-progress, but there's not much I can do
> about the patch comment now. Did you have some followup in mind?

I was thinking about that. I guess a small comment in the tests to
explain what is meant with 'simple' would be fine, something like "#
simple scenario here means we don't remove patches that are part of the
conflict".

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch1915>
__________________________________


More information about the darcs-devel mailing list