-
Notifications
You must be signed in to change notification settings - Fork 295
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 py.typed file #833
Add py.typed file #833
Conversation
Thanks! Could you also add a changelog entry?
To test this, you could run |
Thanks for the quick feedback! Wasn't sure if this was big enough to warrant a changelog entry, but I'll get that fixed and check the zip_safe flag over the weekend :) |
Hey, sorry about the late follow up here. I researched the zip_safe flag and it seems to only be relevant for .egg distraction, which from what I understood is deprecated anyway. Looking into the PRs when the warning was added to mypy it seems like it's not a common issue, so I'm inclined skip that bit. Does that seem reasonable? |
As long as the empty |
It does, running Note that the typing experience with bleak isn't perfect after this, as noted in #822. At least in strict mode mypy will still complain:
|
This is needed for mypy to read the type signatures defined in the project, ref https://mypy.readthedocs.io/en/stable/installed_packages.html#creating-pep-561-compatible-packages
86cf476
to
85b8562
Compare
Thanks! |
Another related discussion on these two types in particular is #582. It is a rather large chore, but would be nice for consuming the API with static analysis tools like mypy and pyright. |
Yeah, I saw that issue and completely agree :) |
This is needed for mypy to read the type signatures defined in the project, ref https://mypy.readthedocs.io/en/stable/installed_packages.html#creating-pep-561-compatible-packages
There's also a note about needing
zip_safe=False
, but I'm not to familiar with setuptools so I didn't want to add that unless it's strictly required.