-
-
Notifications
You must be signed in to change notification settings - Fork 267
Add Matter metadata #3129
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
Add Matter metadata #3129
Conversation
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
#3171 Bundle Size — 11.26MiB (+0.13%).20d839f(current) vs 60d7f8c main#3157(baseline) Warning Bundle contains 2 duplicate packages – View duplicate packages Bundle metrics
Bundle size by type
Bundle analysis report Branch digitaldan:matter-metadata Project dashboard Generated by RelativeCI Documentation Report issue |
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Hi, @florian-h05 this is ready for review when you get the chance, thanks! |
I’ve just started reviewing the binding PR, hope I can finish it today (had to leave my keyboard for now). |
Thanks! No worries if not today ;-) |
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
…imal values Stepsize is 0.01. Signed-off-by: Florian Hotze <dev@florianhotze.com>
Signed-off-by: Florian Hotze <dev@florianhotze.com>
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.
LGTM, thanks!
I have fixed/improved a few things that I noticed, please have a look at my comments and confirm they are correct.
Thanks @florian-h05 for the additions and review! LGTM |
Congrats all around! 💯 |
This is in preparation for the upcoming Matter binding. I'm posting a bit early for visibility, still need to test this quite a bit.
See openhab/openhab-addons#18486