D3870: rebase: add --confirm option

yuja (Yuya Nishihara) phabricator at mercurial-scm.org
Fri Jul 13 08:29:19 EDT 2018


yuja added a comment.


  >   >> +            if confirm:
  >   >>  +                # abort as in-memory merge doesn't support conflict
  >   >>  +                rbsrt._prepareabortorcontinue(isabort=True, backup=False,
  >   >>  +                                              suppwarns=True)
  >   >>  +                needsabort = False
  >   >>  +                if not ui.promptchoice(_(b'apply changes (yn)?'
  >   >>  +                                         b'$$ &Yes $$ &No')):
  >   > 
  >   > Nit: This isn't actually "apply changes".
  >   
  >   Hmm, then what it should be?
  
  I think we'll need to tell that we'll rerun the rebase that's known to have
  at least one merge conflict. "apply changes" sounds like it would just
  finalize the rebase up to the previous revision of the merge conflict.
  
  Alternatively, it can just abort without the prompt since the dry-run was
  incomplete.

REPOSITORY
  rHG Mercurial

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

To: khanchi97, #hg-reviewers
Cc: yuja, mercurial-devel


More information about the Mercurial-devel mailing list