-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Move OnboardingCodesUtil out of app/server #37401
Conversation
a4df653
to
445eb31
Compare
PR #37401: Size comparison from 577b3f2 to 445eb31 Full report (42 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, psoc6, qpg, stm32, telink, tizen)
|
445eb31
to
e9170fb
Compare
0101630
to
66fc382
Compare
66fc382
to
d5184df
Compare
d5184df
to
d52286c
Compare
d52286c
to
1bd0296
Compare
PR #37401: Size comparison from a998bf6 to 3d4746e Full report (61 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, efr32, esp32, linux, nrfconnect, qpg, stm32, telink, tizen)
|
3d4746e
to
8c50292
Compare
09b20d3
to
962ad8f
Compare
e53d023
to
426db4c
Compare
PR #37401: Size comparison from 7adac6e to 426db4c Full report (17 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, qpg, stm32, tizen)
|
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.
Please remove the multiple includes
426db4c
to
ff36e43
Compare
PR #37401: Size comparison from 7adac6e to ff36e43 Full report (83 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
Seems fine, but we should really think about the naming here... Maybe we should have separate headers for the "get a thing" methods and the "print a thing" methods....
* Move OnboardingCodesUtils to separate target. * Remove shell dependency on server. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Use PlatformMgr directly. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
ff36e43
to
4d93f88
Compare
* Move OnboardingCodesUtil out of server * Move OnboardingCodesUtils to separate target. * Remove shell dependency on server. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [mw320] Remove shell dependency on app/server Use PlatformMgr directly. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [NXP][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [linux][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [Infineon][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> --------- Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
* Move OnboardingCodesUtil out of server * Move OnboardingCodesUtils to separate target. * Remove shell dependency on server. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [mw320] Remove shell dependency on app/server Use PlatformMgr directly. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [NXP][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [linux][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [Infineon][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> --------- Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
…ip#37401) * Move OnboardingCodesUtil out of server * Move OnboardingCodesUtils to separate target. * Remove shell dependency on server. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [mw320] Remove shell dependency on app/server Use PlatformMgr directly. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [NXP][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [linux][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [Infineon][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> --------- Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> (cherry picked from commit b959364)
* Move OnboardingCodesUtil out of server * Move OnboardingCodesUtils to separate target. * Remove shell dependency on server. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [mw320] Remove shell dependency on app/server Use PlatformMgr directly. Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [NXP][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [linux][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> * [Infineon][examples] Add missing dependency Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no> --------- Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
Testing
Tested using automated tests.