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

misc: Add module type hints #33

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adamantike
Copy link

This commit adds type hints to the module. It's missing the addition of a py.typed file to be PEP 561 compliant [1]. However, it seems that will require moving away from the py_modules setup in setup.py, according to the PEP:

This PEP does not support distributing typing information as part of module-only distributions or single-file modules within namespace packages.
The single-file module should be refactored into a package and indicate that the package supports typing as described above.

[1] https://peps.python.org/pep-0561/

This commit adds type hints to the module. It's missing the addition of
a `py.typed` file to be PEP 561 compliant [1]. However, it seems that
will require moving away from the `py_modules` setup in `setup.py`,
according to the PEP:

> This PEP does not support distributing typing information as part of
> module-only distributions or single-file modules within namespace
> packages.
> The single-file module should be refactored into a package and
> indicate that the package supports typing as described above.

[1] https://peps.python.org/pep-0561/
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.

1 participant