Skip to content

Conversation

@dhalbert
Copy link
Contributor

@dhalbert dhalbert commented Dec 10, 2025

Pins were in the wrong order. Thanks Jeff M in discord: https://discord.com/channels/327254708534116352/537365702651150357/1448133233647812618

Pins were inadvertently swapped as part of #117.

@dhalbert dhalbert requested review from a team and TheKitty December 10, 2025 12:12
Copy link

@jerryneedell jerryneedell left a comment

Choose a reason for hiding this comment

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

Did not test, but this is obviously correct.

Updated comments for clarity on GPS module connections.
Copy link

@jerryneedell jerryneedell left a 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!

@dhalbert dhalbert merged commit 72d0450 into main Dec 10, 2025
2 checks passed
@dhalbert dhalbert deleted the dhalbert-patch-1 branch December 10, 2025 20:10
adafruit-adabot pushed a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Dec 11, 2025
Updating https://github.com/adafruit/Adafruit_CircuitPython_GPS to 3.11.6 from 3.11.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_GPS#121 from adafruit/dhalbert-patch-1

Updating https://github.com/adafruit/Adafruit_CircuitPython_LSM6DS to 4.6 from 4.5.18:
  > Merge pull request adafruit/Adafruit_CircuitPython_LSM6DS#68 from mikeysklar/low-power

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 3.4.2 from 3.4.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#231 from adafruit/Neradoc-patch-1

Updating https://github.com/adafruit/Adafruit_CircuitPython_PortalBase to 3.4.3 from 3.4.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_PortalBase#119 from makermelissa/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_Qualia to 2.2.0 from 2.1.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Qualia#15 from makermelissa/main
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