marmoute added inline comments.

INLINE COMMENTS

> martinvonz wrote in localrepo.py:2868-2874
> Is this correct when grafting (not merging), such as when rebasing? Is it 
> worth having a test case for that? Is it even possible to get here in that 
> case? Maybe when rebasing a merge commit?

I don't know, but not existing test broke. Consolidating test for the plain 
merge case already took a long time. Not having any test failure from exisitng 
rebase/graft test seemed enough to me while making progress on the correctness 
of this.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D8243/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D8243

To: marmoute, #hg-reviewers
Cc: martinvonz, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to