Skip to content

return bool from read_pin() - #9

Merged
FoamyGuy merged 2 commits into
adafruit:mainfrom
FoamyGuy:read_pin_return_bool
Jan 7, 2025
Merged

return bool from read_pin()#9
FoamyGuy merged 2 commits into
adafruit:mainfrom
FoamyGuy:read_pin_return_bool

Conversation

@FoamyGuy

@FoamyGuy FoamyGuy commented Jan 3, 2025

Copy link
Copy Markdown
Contributor

@ladyada

It was mentioned in adafruit/Adafruit_CircuitPython_PCF8575#6 that this library had the same differing return type. This is the same fix as from that repo.

@tannewt tannewt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One suggestion. Thanks!

Comment thread adafruit_pcf8574.py Outdated
Co-authored-by: Scott Shawcroft <scott@tannewt.org>
@FoamyGuy
FoamyGuy merged commit 1552cb6 into adafruit:main Jan 7, 2025
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jan 8, 2025
Updating https://github.com/adafruit/Adafruit_CircuitPython_LPS2X to 3.0.9 from 3.0.8:
  > Merge pull request adafruit/Adafruit_CircuitPython_LPS2X#17 from jposada202020/adding_displayio_example
  > Merge pull request adafruit/Adafruit_CircuitPython_LPS2X#18 from ncguk/patch-1

Updating https://github.com/adafruit/Adafruit_CircuitPython_NeoPixel to 6.3.13 from 6.3.12:
  > Merge pull request adafruit/Adafruit_CircuitPython_NeoPixel#168 from mattcocca/pixel_order_docs_update

Updating https://github.com/adafruit/Adafruit_CircuitPython_PCF8574 to 1.0.11 from 1.0.10:
  > Merge pull request adafruit/Adafruit_CircuitPython_PCF8574#9 from FoamyGuy/read_pin_return_bool

Updating https://github.com/adafruit/Adafruit_CircuitPython_WS2801 to 1.0.0 from 0.10.19:
  > Merge pull request adafruit/Adafruit_CircuitPython_WS2801#25 from FoamyGuy/use_pixelbuf
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.

3 participants