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.

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

Replace deprecated aliases HomeAssistantType and ServiceCallType #374

Closed
wants to merge 1 commit into from

Conversation

rneubert
Copy link

Fixes #369

HomeAssistantType was used from solis, this is a deprecated alias which will be removed in HA Core 2025.5. Use homeassistant.core.HomeAssistant instead, please report it to the author of the 'solis' custom integration

ServiceCallType was used from solis, this is a deprecated alias which will be removed in HA Core 2025.5. Use homeassistant.core.ServiceCall instead, please report it to the author of the 'solis' custom integration

Fixes hultenvp#369 

HomeAssistantType was used from solis, this is a deprecated alias which will be removed in HA Core 2025.5. Use homeassistant.core.HomeAssistant instead, please report it to the author of the 'solis' custom integration

ServiceCallType was used from solis, this is a deprecated alias which will be removed in HA Core 2025.5. Use homeassistant.core.ServiceCall instead, please report it to the author of the 'solis' custom integration
@@ -8,10 +8,10 @@
import voluptuous as vol
from homeassistant.config_entries import ConfigEntry, SOURCE_IMPORT
from homeassistant.const import Platform, ATTR_NAME, CONF_NAME
from homeassistant.core import HomeAssistant
from homeassistant.core import HomeAssistant, ServiceCall

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, ServiceCall is not used in this file so you can remove this too.

Suggested change
from homeassistant.core import HomeAssistant, ServiceCall
from homeassistant.core import HomeAssistant

@CraigCallender
Copy link

Thanks for this @rneubert. I literally came here to submit my own PR for this and found yours. :)

@jmason
Copy link
Collaborator

jmason commented Jul 11, 2024

Thanks -- this is now merged into #380 as a combined PR.

@hultenvp
Copy link
Owner

As this one is merged via #380 I'll close this PR.

@hultenvp hultenvp closed this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecated alias which will be removed in HA Core 2025.5
5 participants