Skip to content

Deprecate Remote.ls_remotes in favor of Remote.list_heads#1410

Merged
jdavid merged 2 commits into
libgit2:masterfrom
jorio:deprecate-lsremotes
Aug 13, 2025
Merged

Deprecate Remote.ls_remotes in favor of Remote.list_heads#1410
jdavid merged 2 commits into
libgit2:masterfrom
jorio:deprecate-lsremotes

Conversation

@jorio

@jorio jorio commented Aug 13, 2025

Copy link
Copy Markdown
Contributor

This PR implements changes discussed in #1397 (comment)

@jdavid

jdavid commented Aug 13, 2025

Copy link
Copy Markdown
Member

Thanks @jorio
I think ls_remotes should be fully backwards compatible. I mean, it should return None as before instead of the zero Oid (for the loid field).

@jorio jorio force-pushed the deprecate-lsremotes branch from 86715f8 to d16378e Compare August 13, 2025 15:23
@jorio

jorio commented Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

@jdavid That makes sense, please see d16378e

@jdavid jdavid merged commit d16378e into libgit2:master Aug 13, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants