-
Notifications
You must be signed in to change notification settings - Fork 3
changing namespace for timedAttempt to avoid conflicts with time() #23
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
changing namespace for timedAttempt to avoid conflicts with time() #23
Conversation
Memory usage change @ 4ea7d98
Click for full report table
Click for full report CSV
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe using tattempt
instead of timed
would make the difference more clear. I feel like timed
is too much similar to time
i would also add the rp2040 core to the ci to catch the error in this repo https://github.com/arduino-libraries/ArduinoIoTCloud/blob/501271b5a868c178595265d4c83b0777aba8fe28/.github/workflows/compile-examples.yml#L314 |
4ea7d98
to
81c52c1
Compare
Memory usage change @ 81c52c1
Click for full report table
Click for full report CSV
|
I have split your request into #24 |
The error won't be caught with the current examples in cloud utils, since it is required to import the WiFi library in order to have a clash. However this fixes the issue on IoTCloud |
This PR aims to solve the issue appearing on IoTCloud which states
error: reference to 'time' is ambiguous
https://github.com/arduino-libraries/ArduinoIoTCloud/actions/runs/13768398142/job/38502459222?pr=527