Skip to content

Conversation

FoamyGuy
Copy link
Contributor

@FoamyGuy FoamyGuy commented Dec 7, 2021

resolves: #26

The syntax on the typing for one of the arguments was a little different than the others. This PR changes it to match the others and that appears to get everything back to working order.

Tested successfully on Wio Terminal:

Adafruit CircuitPython 7.1.0-beta.1 on 2021-11-30; Seeeduino Wio Terminal with samd51p19
Board ID:seeeduino_wio_terminal

@FoamyGuy FoamyGuy requested a review from a team December 7, 2021 18:45
Copy link
Contributor

@KeithTheEE KeithTheEE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like it matches the typing pattern of the two options below it. I have not tested it but it seems like it should solve it.

@FoamyGuy
Copy link
Contributor Author

FoamyGuy commented Dec 7, 2021

Thanks for looking it over @KeithTheEE!

I'm going to go ahead and merge now to get the library back into a working state. I tested on hardware as well and am quite confident in this fix.

@FoamyGuy FoamyGuy merged commit f7c6c91 into adafruit:main Dec 7, 2021
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Dec 8, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI to 3.5.13 from 3.5.12:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#131 from adafruit/anecdata-patch-1
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#145 from dhalbert/make-package
  > update rtd py version

Updating https://github.com/adafruit/Adafruit_CircuitPython_RTTTL to 2.4.9 from 2.4.8:
  > Merge pull request adafruit/Adafruit_CircuitPython_RTTTL#27 from FoamyGuy/typing_fix
  > update rtd py version

Updating https://github.com/adafruit/Adafruit_CircuitPython_Slideshow to 1.7.2 from 1.7.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_Slideshow#41 from dhalbert/OnDiskBitmap-filename
  > update rtd py version
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.

NameError on Import

2 participants