evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

String formatting and max line length - Part 1 (#84390)

Browse source 
Co-authored-by: Erik Montnemery <erik@montnemery.com>
This commit is contained in:
Franck Nijhof 2022-12-22 10:12:50 +01:00 • committed by GitHub
parent 41341c76cf
commit b0cee0bc46
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
98 changed files with 532 additions and 237 deletions
Download patch file Download diff file Expand all files Collapse all files

52
homeassistant/config_entries.py
View file

@ -278,9 +278,11 @@ class ConfigEntry:
disabled_by, ConfigEntryDisabler
):
report( # type: ignore[unreachable]
"uses str for config entry disabled_by. This is deprecated and will "
"stop working in Home Assistant 2022.3, it should be updated to use "
"ConfigEntryDisabler instead",
(
"uses str for config entry disabled_by. This is deprecated and will"
" stop working in Home Assistant 2022.3, it should be updated to"
" use ConfigEntryDisabler instead"
),
error_if_core=False,
)
disabled_by = ConfigEntryDisabler(disabled_by)
@ -353,7 +355,10 @@ class ConfigEntry:
integration.get_platform("config_flow")
except ImportError as err:
_LOGGER.error(
"Error importing platform config_flow from integration %s to set up %s configuration entry: %s",
(
"Error importing platform config_flow from integration %s to"
" set up %s configuration entry: %s"
),
integration.domain,
self.domain,
err,
@ -410,14 +415,20 @@ class ConfigEntry:
ready_message = f"ready yet: {message}" if message else "ready yet"
if tries == 1:
_LOGGER.warning(
"Config entry '%s' for %s integration not %s; Retrying in background",
(
"Config entry '%s' for %s integration not %s; Retrying in"
" background"
),
self.title,
self.domain,
ready_message,
)
else:
_LOGGER.debug(
"Config entry '%s' for %s integration not %s; Retrying in %d seconds",
(
"Config entry '%s' for %s integration not %s; Retrying in %d"
" seconds"
),
self.title,
self.domain,
ready_message,
@ -1065,8 +1076,9 @@ class ConfigEntries:
if entry.state is not ConfigEntryState.NOT_LOADED:
raise OperationNotAllowed(
f"The config entry {entry.title} ({entry.domain}) with entry_id {entry.entry_id}"
f" cannot be setup because is already loaded in the {entry.state} state"
f"The config entry {entry.title} ({entry.domain}) with entry_id"
f" {entry.entry_id} cannot be setup because is already loaded in the"
f" {entry.state} state"
)
# Setup Component if not set up yet
@ -1090,8 +1102,9 @@ class ConfigEntries:
if not entry.state.recoverable:
raise OperationNotAllowed(
f"The config entry {entry.title} ({entry.domain}) with entry_id "
f"{entry.entry_id} cannot be unloaded because it is not in a recoverable state ({entry.state})"
f"The config entry {entry.title} ({entry.domain}) with entry_id"
f" {entry.entry_id} cannot be unloaded because it is not in a"
f" recoverable state ({entry.state})"
)
return await entry.async_unload(self.hass)
@ -1126,9 +1139,11 @@ class ConfigEntries:
disabled_by, ConfigEntryDisabler
):
report( # type: ignore[unreachable]
"uses str for config entry disabled_by. This is deprecated and will "
"stop working in Home Assistant 2022.3, it should be updated to use "
"ConfigEntryDisabler instead",
(
"uses str for config entry disabled_by. This is deprecated and will"
" stop working in Home Assistant 2022.3, it should be updated to"
" use ConfigEntryDisabler instead"
),
error_if_core=False,
)
disabled_by = ConfigEntryDisabler(disabled_by)
@ -1225,8 +1240,10 @@ class ConfigEntries:
) -> None:
"""Forward the setup of an entry to platforms."""
report(
"called async_setup_platforms instead of awaiting async_forward_entry_setups; "
"this will fail in version 2022.12",
(
"called async_setup_platforms instead of awaiting"
" async_forward_entry_setups; this will fail in version 2022.12"
),
# Raise this to warning once all core integrations have been migrated
level=logging.DEBUG,
error_if_core=False,
@ -1777,7 +1794,10 @@ class EntityRegistryDisabledHandler:
self.changed = set()
_LOGGER.info(
"Reloading configuration entries because disabled_by changed in entity registry: %s",
(
"Reloading configuration entries because disabled_by changed in entity"
" registry: %s"
),
", ".join(to_reload),
)