Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[HASS Deprecation] is_metric deprecated property - 2023.1.0 #214

Closed
bstempien opened this issue Dec 3, 2022 · 3 comments · Fixed by #208
Closed

[HASS Deprecation] is_metric deprecated property - 2023.1.0 #214

bstempien opened this issue Dec 3, 2022 · 3 comments · Fixed by #208
Assignees
@PeteRager
Labels
HA Compliance

Comments

@bstempien
Copy link

Installed 2022.11.5 today and noticed this in my Logs.

Detected integration that accesses the is_metric property of the unit system. This is deprecated and will stop working in Home Assistant 2023.1. Please adjust to use instance check instead.. Please report issue to the custom integration author for lennoxs30 using this method at custom_components/lennoxs30/init.py, line 435: self._is_metric: bool = hass.config.units.is_metricDescribe the bug
@PeteRager
Copy link
Owner

Excellent, thanks for the report. Looks like I have 27 days to fix.

@PeteRager
Copy link
Owner

Example of the fix
ReneNulschDE/mbapi2020@7e97db7

@PeteRager PeteRager changed the title Warning about deprecated property [HASS Deprecation] is_metric Warning about deprecated property Dec 6, 2022
@PeteRager PeteRager changed the title [HASS Deprecation] is_metric Warning about deprecated property [HASS Deprecation] warning is_metric deprecated property - 2023.1.0 Dec 6, 2022
@PeteRager PeteRager changed the title [HASS Deprecation] warning is_metric deprecated property - 2023.1.0 [HASS Deprecation] is_metric deprecated property - 2023.1.0 Dec 6, 2022
@PeteRager PeteRager self-assigned this Dec 6, 2022
PeteRager added a commit that referenced this issue Dec 7, 2022
@PeteRager
#214 - hass unit system deprecation
b0be353 
Pylint
@PeteRager PeteRager linked a pull request Dec 7, 2022 that will close this issue
2022.12.0 release #208
Merged
@PeteRager
Copy link
Owner

Looks like this change was introduced in 2022.11.0.

@PeteRager PeteRager mentioned this issue Apr 19, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PeteRager PeteRager

Labels
HA Compliance
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2022.12.0 release PeteRager/lennoxs30
2 participants
@bstempien @PeteRager