Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a commitish type #578

Open
renatav opened this issue Jan 7, 2025 · 0 comments
Open

Add a commitish type #578

renatav opened this issue Jan 7, 2025 · 0 comments

Comments

@renatav
Copy link
Collaborator

renatav commented Jan 7, 2025

Consider adding a committish type or a hash type so that the type hints are more useful. Right now a lot of strings are passed around, so have to look at code to see what that string is. E g. very simple class Committish(str): pass that would make it a lot clearer what was being returned/consumed.

@renatav renatav assigned renatav and unassigned renatav Jan 7, 2025
@renatav renatav moved this to Backlog in TAF Planning Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog
Development

No branches or pull requests

1 participant