-
Notifications
You must be signed in to change notification settings - Fork 58
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
0.2.11 wheels include typing info #101
Comments
You are right. Currently CI is only used for testing, sdists and wheels are built and uploaded by @jquast. |
Sorry about that, I'll bump and make another release without it, I will be sure to |
0.2.12 released, thanks! |
I record my terminal sessions for important stuff like this and checked it for this release, it did get included in the distribution erroneously during my session, but I was on the master branch at the time it was built and it did not appear in output of git status. I should have also executed git status
buildCommand was
|
While I actually appreciate this because I use typing and it allows me to drop a
#type: ignore
marker when importing the package, based on #71 not being merged, I assume they actually shouldn't be there.The sdist does not have typing info leading me to think the wheel may have been created on a different branch?
Also, the wheel looks to have a spurious
.DS_Store
file, similar to what's mentioned here: pypa/wheel#297I haven't checked the CI yet, so i don't know if you're doing sdist/wheel builds and uploads there, my guess is no, but maybe they can be triggered on tags.
The text was updated successfully, but these errors were encountered: