( )⚙ D11366 dirstate-item: implement `added` in a simpler way

This is an archive of the discontinued Mercurial Phabricator instance.

Differential D11366

dirstate-item: implement `added` in a simpler way
ClosedPublic

Authored by marmoute on Aug 27 2021, 1:11 PM.

Details

Reviewers
Alphare
Group Reviewers
hg-reviewers
Commits
rHG03f57915e0ce: dirstate-item: implement `added` in a simpler way
Summary

We can simply use the underlying attribute.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

marmoute created this revision.Aug 27 2021, 1:11 PM
Alphare accepted this revision.Aug 30 2021, 5:50 AM
This revision is now accepted and ready to land.Aug 30 2021, 5:50 AM
baymax updated this revision to Diff 30143.Aug 30 2021, 9:00 AM

✅ refresh by Heptapod after a successful CI run (🐙 💚)

marmoute added a commit: rHG03f57915e0ce: dirstate-item: implement `added` in a simpler way.Aug 30 2021, 9:05 AM
Closed by commit rHG03f57915e0ce: dirstate-item: implement `added` in a simpler way (authored by marmoute). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mmercurial/pure/parsers.py (2 lines)

Diff 30143

mercurial/pure/parsers.py

@property @property
def tracked(self): def tracked(self):
"""True is the file is tracked in the working copy""" """True is the file is tracked in the working copy"""
return self._wc_tracked return self._wc_tracked
@property @property
def added(self): def added(self):
"""True if the file has been added""" """True if the file has been added"""
return self.v1_state() == b'a' return self._wc_tracked and not (self._p1_tracked or self._p2_tracked)
@property @property
def merged(self): def merged(self):
"""True if the file has been merged """True if the file has been merged
Should only be set if a merge is in progress in the dirstate Should only be set if a merge is in progress in the dirstate
""" """
return self.v1_state() == b'm' return self.v1_state() == b'm'