-
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
Fix incorrect return codes in door lock cluster #19813
Fix incorrect return codes in door lock cluster #19813
Conversation
PR #19813: Size comparison from 39073b3 to c8d50ed Increases above 0.2%:
Increases (16 builds for cc13x2_26x2, cyw30739, esp32, k32w, linux, mbed, nrfconnect, p6)
Decreases (3 builds for cc13x2_26x2)
Full report (36 builds for cc13x2_26x2, cyw30739, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #19813: Size comparison from 5a44f1e to b50805f Increases above 0.2%:
Increases (19 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (3 builds for cc13x2_26x2)
Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #19813: Size comparison from 777d72d to f022314 Increases above 0.2%:
Increases (15 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, mbed, nrfconnect, p6, telink)
Decreases (3 builds for cc13x2_26x2)
Full report (30 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #19813: Size comparison from 1381c8e to 5449e61 Increases above 0.2%:
Increases (19 builds for cc13x2_26x2, cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (5 builds for cc13x2_26x2)
Full report (39 builds for cc13x2_26x2, cyw30739, efr32, k32w, linux, mbed, nrfconnect, p6, telink)
|
Fast tracking, given this has had enough time for review. |
PR #19813: Size comparison from 6e6892a to 9f8a5fa Increases above 0.2%:
Increases (18 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (3 builds for cc13x2_26x2)
Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
PR #19813: Size comparison from 19d0f52 to 6502e58 Increases above 0.2%:
Increases (18 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
Decreases (3 builds for cc13x2_26x2)
Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
6502e58
to
49fe2f3
Compare
c41d9ef
to
72d5531
Compare
PR #19813: Size comparison from 126f6b2 to 9f25124 Increases above 0.2%:
Increases (15 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, mbed, nrfconnect, p6, telink)
Decreases (5 builds for cc13x2_26x2, linux)
Full report (30 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
…schedule operations in the door lock
- Refactor the cluster, so it is consistent in way of handling commands
…into a single function, fix typos
9f25124
to
b6293e8
Compare
b6293e8
to
4864fd2
Compare
PR #19813: Size comparison from 1107d8e to 4864fd2 Increases above 0.2%:
Increases (15 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, mbed, nrfconnect, p6, telink)
Decreases (5 builds for cc13x2_26x2, linux, telink)
Full report (30 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
* [#19622] Return generic failure instead of not found for schedule operations in the door lock * [#19563] Check enum ranges for Door Lock commands - Refactor the cluster, so it is consistent in way of handling commands * Refactoring of the door lock cluster: move credential error checking into a single function, fix typos * Fix tests for Door Lock cluster and run them against the lock app * Update auto-generated files
* [#19622] Return generic failure instead of not found for schedule operations in the door lock * [#19563] Check enum ranges for Door Lock commands - Refactor the cluster, so it is consistent in way of handling commands * Refactoring of the door lock cluster: move credential error checking into a single function, fix typos * Fix tests for Door Lock cluster and run them against the lock app * Update auto-generated files Co-authored-by: Evgeniy Morozov <[email protected]>
Problem
Change overview
Testing