wasilukm / hoymiles-mqtt

Send data from Hoymiles PV plant to Home Assistant without cloud
MIT License
42 stars 8 forks source link

Remove device name from entity name. #17

Closed wasilukm closed 10 months ago

wasilukm commented 10 months ago

Starting from Home Assistant 2.8 entity name shall not include device name (https://developers.home-assistant.io/blog/2023-057-21-change-naming-mqtt-entities/#naming-of-mqtt-entities)

codecov[bot] commented 10 months ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +40.37% :tada:

Comparison is base (3c16129) 0.00% compared to head (6062847) 40.37%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #17 +/- ## ========================================== + Coverage 0.00% 40.37% +40.37% ========================================== Files 5 5 Lines 214 213 -1 Branches 0 34 +34 ========================================== + Hits 0 86 +86 + Misses 214 124 -90 - Partials 0 3 +3 ``` | [Files Changed](https://app.codecov.io/gh/wasilukm/hoymiles-mqtt/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mariusz+Wasiluk) | Coverage Δ | | |---|---|---| | [hoymiles\_mqtt/ha.py](https://app.codecov.io/gh/wasilukm/hoymiles-mqtt/pull/17?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mariusz+Wasiluk#diff-aG95bWlsZXNfbXF0dC9oYS5weQ==) | `69.16% <100.00%> (+69.16%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/wasilukm/hoymiles-mqtt/pull/17/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Mariusz+Wasiluk)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.