D6741: interfaces: create a new folder for interfaces and move repository.py in it
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Tue Aug 20 15:10:36 UTC 2019
durin42 added a comment.
In D6741#99023 <https://phab.mercurial-scm.org/D6741#99023>, @pulkit wrote:
> In D6741#98947 <https://phab.mercurial-scm.org/D6741#98947>, @martinvonz wrote:
>
>>> I was trying to understand current interfaces and write new ones and I realized
>>> we need to improve how current interfaces are organised.
>>
>> And what was the reason we need to improve it? I assume we don't really "need" to change it. Will it somehow help with future patches? Or you just like this structure better?
>
> Looking through Augie's hgit patch, I found we need to add more interfaces and decided to work on adding for `store.basicstore`. I found all the current interfaces in repository.py which has grown very large. I decided to create a new file to have interface for the store class, and maybe a new one for dirstate too, and having them in a separate folder dedicated to interfaces will be nice.
I'm inclined to agree - the repository.py file full of interfaces has gotten crufty. It'd be nice to split it out into more files by topic.
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D6741/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D6741
To: pulkit, indygreg, durin42, martinvonz, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list