-
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
[Telink] Lock app refactoring & fix TC-DRLK-2.10 #30301
[Telink] Lock app refactoring & fix TC-DRLK-2.10 #30301
Conversation
Updated lock app to latest cluster, passes TC-DRLK-2.10 Signed-off-by: Misha Tkachenko <[email protected]>
Added storage to save user credentials, fixed test DRLK-2.10 Signed-off-by: Misha Tkachenko <[email protected]>
Updated code style and configs in Lock App
Updated lock app definitions
PR #30301: Size comparison from 42c3af2 to 087831f Full report (56 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
Excluded retention build from Lock App CI due to lack of retention RAM
PR #30301: Size comparison from 42c3af2 to 9c0a8b6 Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
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.
Rubber-stamping platform-specific fix. I skimmed the new LockManager briefly, and it generally looks sane, but did not check all the details.
PR #30301: Size comparison from 512fa70 to b360807 Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Change overview