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 Missing Type Annotations #30
Conversation
submitted for review / comment PS> I think the yield in line 779 should return an Iterator[Tuple[int, int]] but that is breaking the precommits, so maybe not... |
Looks like this just fell through the cracks. I'll request a review and try to get to it shortly if no one else does first. |
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.
Looks good to me, thanks @tcfranks!
I don't have the hardware for testing, but don't spot anything during review that would change behavior.
Nothing you did caused the delay, this one just fell through the cracks and tekktrik and I have been getting into some other things lately.
I'm still in the process of making a sweep through open PRs today (with an intermission for the meeting). If you've got others that are waiting on us feel free to ping me on them and I'll bump them up my list for this afternoon.
Updating https://github.com/adafruit/Adafruit_CircuitPython_MLX90640 to 1.2.14 from 1.2.13: > Merge pull request adafruit/Adafruit_CircuitPython_MLX90640#30 from tcfranks/main > Add .venv to .gitignore Updating https://github.com/adafruit/Adafruit_CircuitPython_Wiznet5k to 1.12.16 from 1.12.15: > Merge pull request adafruit/Adafruit_CircuitPython_Wiznet5k#70 from BiffoBear/add_typing > Add .venv to .gitignore Updating https://github.com/adafruit/Adafruit_CircuitPython_HTTPServer to 1.0.1 from 1.0.0: > Merge pull request adafruit/Adafruit_CircuitPython_HTTPServer#27 from spovlot/response-header-overwrite > Add .venv to .gitignore Updating https://github.com/adafruit/Adafruit_CircuitPython_Logging to 5.1.0 from 5.0.1: > Merge pull request adafruit/Adafruit_CircuitPython_Logging#44 from vladak/multiple_handlers > Add .venv to .gitignore Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA: > Updated download stats for the libraries
resolves #26
submitted for review / comment