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

Type hinting #176

Open
manulera opened this issue Jan 16, 2024 · 2 comments
Open

Type hinting #176

manulera opened this issue Jan 16, 2024 · 2 comments

Comments

@manulera
Copy link
Collaborator

@BjornFJohansson what we discussed today. Once we have done the big merge, I will do a PR adding type hinting to at least some of the classes. This can be very useful for newcomers, to know exactly what the outputs of functions will be, and get intellisense from IDEs.

@manulera manulera self-assigned this Jan 16, 2024
@manulera
Copy link
Collaborator Author

@JeffXiePL will also give a go at this @BjornFJohansson.

I have been looking around for libraries to check types dynamically while running the tests. It seems that typeguard would be a good choice, but I found some problems when trying to implement it. I will see if I figure it out, otherwise we can also not check.

manulera added a commit that referenced this issue Aug 12, 2024
@dgruano
Copy link
Contributor

dgruano commented Sep 5, 2024

Don't hesitate to use Copilot to help you with this, so far I've had good experiences with it

BjornFJohansson pushed a commit that referenced this issue Sep 6, 2024
manulera added a commit that referenced this issue Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants