-
Notifications
You must be signed in to change notification settings - Fork 16
Use circuitpython_typing.FillBasedLED instead of custom Pixel protocol #56
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
Conversation
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 @tekktrik
(noticed actions didn't pass after I posted this)
| from typing import Protocol | ||
| except ImportError: | ||
| from typing_extensions import Protocol | ||
| from adafruit_espatcontrol.adafruit_espatcontrol import ESP_ATcontrol |
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.
I think we need to keep this ESP_ATcontrol line
FoamyGuy
left a comment
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 @tekktrik
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP_ATcontrol to 0.6.7 from 0.6.4: > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#56 from tekktrik/doc/use-circuitpython-typing > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#55 from tekktrik/dev/allow-37-protocol > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#54 from tekktrik/dev/swap-import-order > change discord badge > Patch: Replaced discord badge image > Updated gitignore Updating https://github.com/adafruit/Adafruit_CircuitPython_BitmapSaver to 1.2.0 from 1.1.9: > Merge pull request adafruit/Adafruit_CircuitPython_BitmapSaver#22 from matt-land/fix-variable-reference > change discord badge > Patch: Replaced discord badge image > Update .gitignore > Update Black to latest. > Fixed readthedocs build > Consolidate Documentation sections of README
Pins to
circuitpython_typingminimum release that adds this functionality