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 typehints and return types to private functions and constructors. #242
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
9283035
to
4fecfd9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general those changes look safe. Just some minor things to change.
@Tobion thank you very much for the review... before we proceed with current PR maybe we should have a look on this PR #250 ? It is related to this discussion... after that if someone could merge master branch to |
@Tobion can you check if the applied changes fulfill your requests? |
3a7d0aa
to
47d2d61
Compare
47d2d61
to
eb6f13a
Compare
@gmponos Thanks for working on this. I rebased and finished the PR. |
The process of adding return types and typehinting the code can be a long one.
That's why I thought about starting with the easy ones. So I added return types to private functions and constructors.