-
Notifications
You must be signed in to change notification settings - Fork 8
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 type hints #19
base: main
Are you sure you want to change the base?
Add type hints #19
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.
smdh
@@ -26,7 +26,7 @@ like Sphinx encourages. | |||
|
|||
## [Typehints](https://docs.python.org/3/library/typing.html) | |||
|
|||
* Do not use typehints. | |||
* Please use typehints. |
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.
Did you run |
d7efe95
to
b421312
Compare
Not compatible with the latest |
b421312
to
d483a13
Compare
Rebased the PR on |
In a way it's quite funny:
|
The use of the return code to indicate the number of type annotations is inspired! I think this might be check mate so feel free to close. |
Just for fun. Won't be offended if this isn't merged.
NB type hints allowed for the identification of interesting potential issues e.g.
Element.text
can returnNone
.