D821: unamend: move fb extension unamend to core
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Mon Nov 13 17:25:46 EST 2017
durin42 added inline comments.
> +def unamend(ui, repo, **opts):
> + """undo the amend operation on a current changeset
undo the _most recent_ amend? or can I run this iteratively and undo many amends in sequence?
> + markers = list(predecessormarkers(curctx))
> + if len(markers) != 1:
> + e = _("changeset must have one predecessor, found %i predecessors")
Should we also look for unamend_source in the extra, and potentially refuse to unamend an unamend? Or not?
> + if curctx.children():
> + raise error.Abort(_("cannot unamend in the middle of a stack"))
cannot unamend a changeset with children
To: pulkit, #hg-reviewers, durham
Cc: durin42, ryanmce, singhsrb, durham, mercurial-devel
More information about the Mercurial-devel