-
Notifications
You must be signed in to change notification settings - Fork 3k
mbed.h includes removed #9210
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
mbed.h includes removed #9210
Conversation
@dannybenor @donatieng @artokin @AriParkkila Please review your part of the changes. |
Adding @SeppoTakalo to review nanostack (mbed-mesh-api/interface) related changes. |
@AnttiKauppila, thank you for your changes. |
@OPpuolitaival When this gets merged in, your team can update CI to check that mbed.h is not included in upcoming PRs. |
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.
Looks good to me, except for small changes suggested
@deepikabhavnani Thanks for your feedback. Valid points but whole file seems to be a bit odd. |
CI started |
Test run: FAILEDSummary: 3 of 7 test jobs failed Failed test jobs:
|
Close! @AnttiKauppila Please check the build errors with the |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
One more OK from one of the following teams, and I think we'll be good to merge this: |
FYI AnttiKauppila#7 - this fixes compilation issues with BLE files impacted by this PR |
BLE fixes for ARMmbed#9210
Thanks @donatieng 👍 |
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Removed internal includes of mbed.h from code and added missing headers.
Test, unittest, Target and unsupported folders were not touched.
Pull request type
Reviewers