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

fix for Arduino > 3.1.0 #51

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Conversation

mhaberler
Copy link

client:connect(IPAddress ip, uint16_t port, int32_t timeout) and connect(const char * host, uint16_t port, int32_t timeout)

were added to Client in 3.1.x - handle those

see https://github.com/mhaberler/aptest for a demo using pioarduino 3.0.7 and 3.1.0-rc3

see https://discord.com/channels/1263397951829708871/1268903464700739706 for the release history

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@mlesniew mlesniew force-pushed the arduino-3.1.0 branch 5 times, most recently from 6fab5ba to 7ae9f3b Compare December 4, 2024 18:41
This adds two new method overrides for classes inheriting from ::Client
* connect(IPAddress ip, uint16_t port, int32_t timeout)
* connect(const char * host, uint16_t port, int32_t timeout)

Changes also include a CI update to build tests with both pioarduino and
the official platformio esp32 platform.
Copy link

sonarcloud bot commented Dec 4, 2024

@mlesniew
Copy link
Owner

mlesniew commented Dec 4, 2024

I have polished the changes a bit and added a CI config to build examples with the recent release of pioarduino too.

Unfortunately, this revealed that PicoWebsocket was incompatible, so changes were required there too, see: mlesniew/PicoWebsocket@1664f81

But finally I got it working in all configurations and in the CI.

Thanks for the contribution!

@mlesniew mlesniew merged commit af9d9e7 into mlesniew:master Dec 4, 2024
6 checks passed
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.

2 participants