-
Notifications
You must be signed in to change notification settings - Fork 5
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
typing: add py.typed #13
Conversation
Reproducible on local, python 3.12.7:
|
Looks like sphinx-rtd-theme needs to get locked: https://github.com/adafruit/Adafruit_Board_Toolkit/actions/runs/11788475185/job/32835642010 |
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.
Don't do this by rolling Python or sphinx-rtd-theme
back. Roll pylint and other things forward. Example: https://github.com/adafruit/Adafruit_CircuitPython_HID/blob/main/.pre-commit-config.yaml.
We don't use py.typed anywhere else, including with other libraries that use mypy. Is this historical?
The py.typed file appears to be a current requirement of the spec: https://typing.readthedocs.io/en/latest/spec/distributing.html#packaging-type-information |
You can just remove |
Looks like it's just the docs problem:
This can be resolved be removing the -W flag, locking sphinx-rtd-theme, or addressing the deprecation warning:
Which do you prefer? |
Probably a good idea to compare the ancillary files here with any library that has a had a recent release. |
Some discussion of |
Well, not all Python devs are as ... scrupulous ... as I am 🍸 . |
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! Fine to take this for now while we study py.typed
. Mainly we needed to update the build, as you found out.
I was not aware of |
Closes #12.