D821: unamend: move fb extension unamend to core
pulkit (Pulkit Goyal)
phabricator at mercurial-scm.org
Fri Nov 17 15:06:20 EST 2017
pulkit added inline comments.
> durin42 wrote in uncommit.py:265
> Should we also look for unamend_source in the extra, and potentially refuse to unamend an unamend? Or not?
Okay while trying to add this condition, I found we cannot refuse to unamend a changeset on the basis of unamend_source, for e.g
`a -amend-> b -unamend-> a' -amend-> c -unamend-> a''`
But if we refuse on basis on unamend_source, unamend `c` will refuse. We need to be more smart here but when I think the other way around, I think it's okay to unamend an unamend. I am open to suggestion and will go with what you guys prefer is good.
To: pulkit, #hg-reviewers, durham
Cc: quark, durin42, ryanmce, singhsrb, durham, mercurial-devel
More information about the Mercurial-devel