-
-
Notifications
You must be signed in to change notification settings - Fork 243
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 HomeKit metadata definitions #2691
Conversation
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Signed-off-by: Cody Cutrer <cody@cutrer.us>
Inverted param, enums for all characteristics (including on the accessory itself if it makes sense), configuration options, and metadata characteristics Signed-off-by: Cody Cutrer <cody@cutrer.us>
7f35b8b
to
52e94c3
Compare
You may want to look at the commits individually, since one of them sorts them all. cc @yfre |
#2153 Bundle Size — 10.78MiB (+0.13%).Warning Bundle contains 2 duplicate packages – View duplicate packages Bundle metrics
|
Current #2153 |
Baseline #2150 |
|
---|---|---|
Initial JS | 1.88MiB |
1.88MiB |
Initial CSS | 607.91KiB |
607.91KiB |
Cache Invalidation | 18.11% |
18.11% |
Chunks | 223 |
223 |
Assets | 246 |
246 |
Modules | 2888 |
2888 |
Duplicate Modules | 149 |
149 |
Duplicate Code | 1.84% |
1.84% |
Packages | 97 |
97 |
Duplicate Packages | 2 |
2 |
Bundle size by type 1 change
1 regression
Current #2153 |
Baseline #2150 |
|
---|---|---|
JS | 8.97MiB (+0.15% ) |
8.96MiB |
CSS | 892.47KiB |
892.47KiB |
Fonts | 526.1KiB |
526.1KiB |
Media | 295.6KiB |
295.6KiB |
IMG | 140.74KiB |
140.74KiB |
HTML | 1.24KiB |
1.24KiB |
Other | 871B |
871B |
Bundle analysis report Branch ccutrer:homekit-metadata-update Project dashboard
Signed-off-by: Cody Cutrer <cody@cutrer.us>
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 for the updates!
Let‘s wait a few days for @yfre so he can have a look if he wants to. |
Let's merge this now. |
Add missing accessories, configuration parameters, enums, etc.