Skip to content
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

Switch to using ConnectionManager #197

Merged
merged 14 commits into from Feb 29, 2024

Conversation

justmobilize
Copy link
Contributor

@justmobilize justmobilize commented Dec 22, 2023

Update adafruit_requests to use Adafruit_CircuitPython_ConnectionManager for socket handling. As well as clean up examples

requirements.txt Outdated Show resolved Hide resolved
@justmobilize justmobilize marked this pull request as ready for review February 27, 2024 13:27
Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

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

@justmobilize This looks OK - did you test any of the tests on physical hardware, or on CPython?

conftest.py Outdated Show resolved Hide resolved
@justmobilize
Copy link
Contributor Author

@justmobilize This looks OK - did you test any of the tests on physical hardware, or on CPython?

@brentru I've tested on my m4-express with both a esp32spi and wiznet5k and esp32s3_tft. I used the code from here under MQTT example

@brentru
Copy link
Member

brentru commented Feb 29, 2024

@justmobilize Thanks for testing on hardware. Marking this as approved!

@brentru brentru self-requested a review February 29, 2024 19:26
@brentru brentru merged commit 5462177 into adafruit:main Feb 29, 2024
1 check passed
@justmobilize justmobilize deleted the connection-manager branch February 29, 2024 19:27
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Mar 1, 2024
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP_ATcontrol to 0.11.0 from 0.10.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#70 from justmobilize/remove-set-socket

Updating https://github.com/adafruit/Adafruit_CircuitPython_MiniMQTT to 7.6.0 from 7.5.9:
  > Merge pull request adafruit/Adafruit_CircuitPython_MiniMQTT#197 from justmobilize/connection-manager

Updating https://github.com/adafruit/Adafruit_CircuitPython_PyCamera to 1.2.0 from 1.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_PyCamera#31 from FoamyGuy/overlay_pos

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries
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.

None yet

3 participants