Skip to content
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

Pass zip_safe=False to setup() #151

Merged
merged 1 commit into from
Oct 26, 2021
Merged

Conversation

bcb
Copy link
Contributor

@bcb bcb commented Oct 26, 2021

I noticed py.typed is not being installed with this library, which makes mypy complain that the library is not typed.

The file is included in package_data, but you also need to add zip_safe=False in order for this to work - see mypy docs.

Screen Shot 2021-10-26 at 2 03 09 pm

@ilanschnell
Copy link
Owner

Thanks for the PR! Good catch. I thought that since the package contains C extensions, it wouldn't be zip-safe anyway, but it's better to explicit.

@ilanschnell ilanschnell merged commit d05dd66 into ilanschnell:master Oct 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants