( )⚙ D6884 bookmarks: remove changectx() method from bmstore (API)

This is an archive of the discontinued Mercurial Phabricator instance.

bookmarks: remove changectx() method from bmstore (API)
ClosedPublic

Authored by durin42 on Sep 25 2019, 6:14 PM.

Details

Summary

All the callsites of this method have access to the repo, and I'd
rather not have to duplicate this across alternative bmstore
implementations. Besides, it feels like a bit of a layering violation.

.. api::

`mercurial.bookmarks.bmstore` no longer has a convenience method
for looking up changectx instances from a bookmark name. Use
`repo[repo.bookmarks[name]]` intead of
`repo.bookmarks.changectx(name)`.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped