-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Add binaryornot stubs #13920
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 binaryornot stubs #13920
Conversation
This comment has been minimized.
This comment has been minimized.
3b2a117
to
ec07240
Compare
This comment has been minimized.
This comment has been minimized.
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.
Thanks! Remarks below.
This comment has been minimized.
This comment has been minimized.
According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉 |
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.
Thanks! For future reference: Please don't force push, this makes reviewing considerably harder.
https://github.com/binaryornot/binaryornot
Upstream has been inactive for 2 months and has a PR open to add a minimal type stubs (in the code would be better), but it is incomplete.
helpers.is_binary_string
does not acceptstr
(as the name may suggest) because it calls onchardet.detect()
which only acceptsbytes | bytearray
.