D1047: tweakdefaults: add restack command

quark (Jun Wu) phabricator at mercurial-scm.org
Fri Oct 13 15:08:06 EDT 2017


quark added a comment.


  In https://phab.mercurial-scm.org/D1047#17639, @lothiraldan wrote:
  
  > But instead of having an alias, no matter how awesome it is that we can implement such command using only revsets, could we have it instead as a command? So we could add documentation and a proper help. I'm not sure that all end-users will be able to understand the revsets and I don't think the tweakdefaults comments will be available to them.
  
  
  I found the revset helpful for understanding what it does. I understand non-power users may feel differently.
  
  > I think that there were some discussions about some problems with pruned changeset, is there a scenario where restack fails or misbehave with pruned changesets?
  
  If you read the comment or the test, it's already handled by using `first(A+B)`.
  
  > Also as a user, I would like to restack only my current stack (no matter how it is defined) if I have other orphans changesets (like if parts of my changesets have been queued), I don't want them to be rebased while I'm working on something else.
  
  That's doable by changing `-r`.
  
  > With all these considerations, I would prefer to have restack as an experimental extension and not activated by default. We highlight it during 4.4 announcement so people could test with their own workflow and gives feedback before activating it by default.
  
  That sounds okay. I'll move the code to `hgext`.

REPOSITORY
  rHG Mercurial

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

To: quark, #hg-reviewers
Cc: martinvonz, lothiraldan, dlax, mercurial-devel


More information about the Mercurial-devel mailing list