-
Notifications
You must be signed in to change notification settings - Fork 49
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
Better type hints #927
Comments
@Ipuch I got this. You want to change in every python file? |
@EastWest14 Could you first create a PR for a single file with the proposed naming for the all (or most of) the list/tuple (and other typing names that you can think of)? Once we agreed with the naming with @Ipuch , you could change it everywhere. This new file should be misc/types.py Thanks a lot! |
@pariterre @Ipuch Here is a draft for one tiny file: Where should I put the file with all the type names? |
@pariterre @Ipuch Do you want me to continue with PRs for other modules? Happy to keep working on this |
Definitely :) |
That's a great idea ! Great work by the way ;) |
Ok, there are about 6 big modules - I'll be going one by one. Will take about 2-3 days per module. I found it in Github search for open issue and "good first issue". |
With @pariterre, we think that we should migrate to global type hints:
currently we have such hints:
Instead, we would like:
The text was updated successfully, but these errors were encountered: