D6487: states: created new class to handle multi-step operation states

taapas1128 (Taapas Agrawal) phabricator at mercurial-scm.org
Fri Jun 7 15:52:24 EDT 2019


taapas1128 requested review of this revision.
taapas1128 added a comment.


  https://phab.mercurial-scm.org/D6487 : A new `statecheck` class is created with its utility functions.
  https://phab.mercurial-scm.org/D6488 : A new list of `statecheck` objects is created and `checkunfinished` and `clearunfinished` are imported and modified in accordance to the class and the new list
  https://phab.mercurial-scm.org/D6494 : `cmdutil.checkunfinished` and `cmdutil.clearunfinished` is removed . Throughout core and extensions they are replaced by `statemod.checkunfinished` and `statemod.clearunfinished`. Also, the previous appends in extensions that were made to `cmdutil.unfinishedstates` are now made to `statemod.unfinishedstates` as `statecheck` objects. Tests results are stated
  https://phab.mercurial-scm.org/D6495 : Support for the functionalities is added to`cmdutil.STATES` is added to `statemod.unfinishedstates`. So `cmdutil.STATES` is completely removed . Tests results are stated.

REPOSITORY
  rHG Mercurial

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

To: taapas1128, #hg-reviewers, martinvonz
Cc: martinvonz, pulkit, mercurial-devel


More information about the Mercurial-devel mailing list