Page MenuHomePhabricator
Differential D6502

state: moved cmdutil.STATES and utilities to state.py
AcceptedPublic
Actions

Authored by taapas1128 on Sat, Jun 8, 4:57 PM.

Details

Reviewers
pulkit
martinvonz
Group Reviewers
hg-reviewers
Summary

This commit moves cmdutil.STATES and adjoining functions to
state.py. The existing users are updated accordingly.

Tests remain unchanged.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

taapas1128 created this revision.Sat, Jun 8, 4:57 PM
martinvonz added a subscriber: martinvonz.Sat, Jun 8, 6:15 PM
martinvonz added inline comments.
mercurial/state.py
124–125

Nit: This should be moved to the patch that introduced the bad formatting.

134

This feels a lot like morestatus functionality and should perhaps have been left in cmdutil.py.

taapas1128 updated this revision to Diff 15401.Sun, Jun 9, 8:31 AM
taapas1128 updated this revision to Diff 15404.Sun, Jun 9, 8:53 AM
taapas1128 added inline comments.Sun, Jun 9, 8:58 AM
mercurial/state.py
134

morestatus functionality is left in cmdutil.py only the place where it checks for unfinished states that is _morestatus has been moved here.

134

_getrepostate not more status

taapas1128 updated this revision to Diff 15425.Mon, Jun 10, 1:54 PM
taapas1128 updated this revision to Diff 15439.Tue, Jun 11, 9:48 AM
taapas1128 updated this revision to Diff 15515.Fri, Jun 14, 3:28 PM
pulkit accepted this revision as: pulkit.Fri, Jun 14, 6:48 PM
taapas1128 updated this revision to Diff 15528.Sat, Jun 15, 6:39 AM
taapas1128 updated this revision to Diff 15536.Mon, Jun 17, 7:36 AM
taapas1128 updated this revision to Diff 15552.Mon, Jun 17, 2:08 PM
martinvonz added inline comments.Tue, Jun 18, 11:29 AM
mercurial/cmdutil.py
623

Should drop the _ from _getrepostate since it's no longer private.

taapas1128 marked an inline comment as done.Tue, Jun 18, 11:46 AM
taapas1128 updated this revision to Diff 15579.
martinvonz accepted this revision.Tue, Jun 18, 11:48 AM
This revision is now accepted and ready to land.Tue, Jun 18, 11:48 AM
taapas1128 updated this revision to Diff 15583.Tue, Jun 18, 1:05 PM
taapas1128 updated this revision to Diff 15640.Mon, Jun 24, 5:33 AM

Revision Contents
Changeset List

PathPackages
Mmercurial/cmdutil.py (57 lines)
Mmercurial/state.py (61 lines)

Diff 15640

View Options

mercurial/cmdutil.py

Loading...
View Options

mercurial/state.py

Loading...