-
Notifications
You must be signed in to change notification settings - Fork 3k
TARGET_WICED updated. #8949
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
TARGET_WICED updated. #8949
Conversation
Travis failure will be fixed on master soon (we will request rebase once done). See #8945 |
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.
Too fast to approve. Can you add details to the commit message (what is being updated - what this commit brings? ). The description from here would be a good start
The fix is on master, if you rebase, it should allow travis to run |
Connection status issue corrected. If the link layer status changes, the connection status callback is now called correctly. mbedtls headers from library are removed and the mbed-os one's are used.
56d7fa5
to
f6e16b9
Compare
CI started |
Test run: SUCCESSSummary: 4 of 4 test jobs passed |
Just wanted to say, thank you for the details in the commit! |
Description
Connection status issue corrected.
If the link layer status changes, the connection status callback is now called correctly.
Changes have been tested locally and at CI.
Pull request type