-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Update CommissionerControlCluster xml definition to align with spec #35272
Conversation
Review changes with SemanticDiff. Analyzed 5 of 19 files. Overall, the semantic diff is 1% smaller than the GitHub diff.
|
PR #35272: Size comparison from c14a370 to 3a88fed Full report (61 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
I am assuming update to |
The changes to .cpp to stop using the now-nonexistent fields have to happen in this PR to pass CI. |
PR #35272: Size comparison from c14a370 to 1311f3b Full report (77 builds for bl602, bl702, bl702l, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
…roject-chip#35272) * Update CommissionerControlCluster xml defination to align with spec * Clean up commissioner-control-server
Update CommissionerControlCluster xml definition to align with spec change:
https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/10342