-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Restructure netsocket directory #13335
Conversation
1ca7f98
to
de843e0
Compare
3881d21
to
d73a37b
Compare
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.
LGTM otherwise
UNITTESTS/features/cellular/framework/AT/at_cellularcontext/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/AT/at_cellulardevice/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/AT/at_cellularinformation/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/AT/at_cellularnetwork/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/AT/at_cellularsms/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/AT/at_cellularstack/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/device/cellularcontext/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/device/cellulardevice/unittest.cmake
Outdated
Show resolved
Hide resolved
UNITTESTS/features/cellular/framework/device/cellularstatemachine/unittest.cmake
Outdated
Show resolved
Hide resolved
connectivity/netsocket/tests/UNITTESTS/netsocket/CellularNonIPSocket/unittest.cmake
Outdated
Show resolved
Hide resolved
62492b0
to
a4af6e9
Compare
This PR cannot be merged due to conflicts. Please rebase to resolve them. |
Commit the reviewer suggestions Co-authored-by: Lingkai Dong <lingkai.dong@arm.com>
a4af6e9
to
a4c1071
Compare
@0xc0170 This PR is ready for CI. Could you start the CI, please |
CI started |
Test run: FAILEDSummary: 2 of 3 test jobs failed Failed test jobs:
|
Hi, @rajkan01 @paul-szczepanek-arm as I mentioned, in slack, earlier. this PR is dependent on ARMmbed/esp32-driver#4 |
@0xc0170 Please restart the CI for this PR as ARMmbed/esp32-driver#4 merged already that will break CI without this PR |
CI restarted |
Test run: SUCCESSSummary: 6 of 6 test jobs passed |
@0xc0170 This PR CI passed, could you merge |
Summary of changes
Dependent PR: ARMmbed/esp32-driver#4
Restructured
features/netsocket
as per the new directory structure proposal:Impact of changes
None.
Migration actions required
None.
Documentation
To Be Updated.
Pull request type
Test results
Manual testing: (Build for K64F target with GCC_ARM toolchain)
Reviewers
@0xc0170 @ARMmbed/mbed-os-core