-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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(ESP-NOW): Remove all peers on ESP_NOW.end() #10102
fix(ESP-NOW): Remove all peers on ESP_NOW.end() #10102
Conversation
👋 Hello P-R-O-C-H-Y, we appreciate your contribution to this project! Click to see more instructions ...
Review and merge process you can expect ...
|
4488c33
to
fad4391
Compare
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
|
Test Results 56 files - 83 56 suites - 83 4m 53s ⏱️ - 1h 37m 53s Results for commit 7f98186. ± Comparison against base commit 23b84e5. This pull request removes 9 tests.
♻️ This comment has been updated with latest results. |
Description of Change
This PR fixes ESP-NOW library, where on end() peers were not removed correctly.
Tests scenarios
Locally with ESP-NOW examples.
Related links
Related #10076