This is an archive of the discontinued Mercurial Phabricator instance.

repo: remove now-unused changectx() method (API)
ClosedPublic

Authored by martinvonz on Apr 3 2018, 12:14 PM.

Details

Summary

repo.changectx(x) was just a synonym for repo[x], so any extensions
that fail due to this commit should switch over to that form.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

martinvonz created this revision.Apr 3 2018, 12:14 PM
pulkit added a subscriber: pulkit.Apr 3 2018, 2:21 PM

I know it's pretty obvious to us that repo.getitem should be used instead but I will like to have that in commit description so when we look back, or someone updates their extensions, some time can be saved.

martinvonz edited the summary of this revision. (Show Details)Apr 3 2018, 4:19 PM
In D3037#49157, @pulkit wrote:

I know it's pretty obvious to us that repo.getitem should be used instead but I will like to have that in commit description so when we look back, or someone updates their extensions, some time can be saved.

Makes sense. Done.

pulkit accepted this revision.Apr 4 2018, 6:43 AM
This revision was automatically updated to reflect the committed changes.