Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Type hints #108

Answered by Byron
lunacd asked this question in Q&A
Oct 27, 2023 · 1 comments · 1 reply
Discussion options

Would this library be interested in type hints being added? Is there a reason GitPython library have all type hints ready but this part?

If people are open to type hints, I would be glad to help add those.

You must be logged in to vote

Ideally, gitdb wouldn't be used anymore, even from GitPython. Apparently that's not the case in practice though so having types that make it easier to work with seems useful nonetheless.

My recommendation is to restrict any contributed types, which are welcome, to the ones ordinary users would encounter while interacting with GitPython and its default object backend (the one that uses git), and avoid putting any work in types that are related to gitdb being used as pure object database as that shouldn't be done. See #106 as example for how broken it is, besides its sluggish performance.

Replies: 1 comment · 1 reply

Comment options

Ideally, gitdb wouldn't be used anymore, even from GitPython. Apparently that's not the case in practice though so having types that make it easier to work with seems useful nonetheless.

My recommendation is to restrict any contributed types, which are welcome, to the ones ordinary users would encounter while interacting with GitPython and its default object backend (the one that uses git), and avoid putting any work in types that are related to gitdb being used as pure object database as that shouldn't be done. See #106 as example for how broken it is, besides its sluggish performance.

You must be logged in to vote
1 reply
@lunacd
Comment options

Thank you for explaining the situation. If the ideal situation is to sunset this, I probably wouldn't try to add any hint to this.

Answer selected by lunacd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
🙏
Q&A
Labels
None yet
2 participants
Converted from issue

This discussion was converted from issue #107 on October 27, 2023 05:48.

Morty Proxy This is a proxified and sanitized view of the page, visit original site.