-
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
Remove invalid endpoint composition from lighting-app #27133
Remove invalid endpoint composition from lighting-app #27133
Conversation
Problem: - lighting-app example has Scenes and Groups clusters on Endpoint 0, which is invalid. Fixes: - Remove clusters that don't belong on Endpoint 0 - LevelControl - OnOff - Groups - Scenes Testing done: - TC_RR_1_1 still passes - TC_LVL_2_1 still passes
PR #27133: Size comparison from dd21e4e to 2c7c12d Increases (15 builds for bl602, cc32xx, cyw30739, esp32, psoc6, qpg, telink)
Decreases (16 builds for bl702, cyw30739, efr32, k32w, linux, psoc6, telink)
Full report (53 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, 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.
I should note, though, that:
- Except for Groups these were all disabled, so not actually present on the endpoint.
- For groups, I don't recall where the argument about whether EP0 can be in a group ended up.
@tcarmelveilleux now that matter-lint supports these, should the group check become a default reject in our linter rules? |
Scenes yes, group not yet... |
Problem:
Fixes:
Testing done: