Make _LOGGER public in Comelit (#177382)

homeassistant/components/comelit/__init__.py

@@ -6,7 +6,7 @@ from homeassistant.const import CONF_HOST, CONF_PIN, CONF_PORT, CONF_TYPE, Platf
 from homeassistant.core import HomeAssistant, callback
 from homeassistant.helpers import device_registry as dr, entity_registry as er
 
-from .const import _LOGGER, CONF_VEDO_PIN, DEFAULT_PORT, DOMAIN
+from .const import CONF_VEDO_PIN, DEFAULT_PORT, DOMAIN, LOGGER
 from .coordinator import (
     ComelitBaseCoordinator,
     ComelitConfigEntry,
@@ -104,7 +104,7 @@ async def async_migrate_entry(
             ):
                 return None
 
-            _LOGGER.debug(
+            LOGGER.debug(
                 "Migrating from version %s.%s",
                 config_entry.version,
                 config_entry.minor_version,
@@ -119,7 +119,7 @@ async def async_migrate_entry(
 
         hass.config_entries.async_update_entry(config_entry, version=1, minor_version=2)
 
-        _LOGGER.info(
+        LOGGER.info(
             "Migration to version %s.%s successful",
             config_entry.version,
             config_entry.minor_version,

homeassistant/components/comelit/alarm_control_panel.py

@@ -1,6 +1,5 @@
 """Support for Comelit VEDO system."""
 
-import logging
 from typing import TYPE_CHECKING, cast, override
 
 from aiocomelit.api import ComelitVedoAreaObject
@@ -16,13 +15,12 @@ from homeassistant.core import HomeAssistant
 from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
 
+from .const import LOGGER
 from .coordinator import ComelitConfigEntry, ComelitSerialBridge, ComelitVedoSystem
 
 # Coordinator is used to centralize the data updates
 PARALLEL_UPDATES = 0
 
-_LOGGER = logging.getLogger(__name__)
-
 AWAY = "away"
 DISABLE = "disable"
 HOME = "home"
@@ -120,7 +118,7 @@ class ComelitAlarmEntity(
     def alarm_state(self) -> AlarmControlPanelState | None:
         """Return the state of the alarm."""
 
-        _LOGGER.debug(
+        LOGGER.debug(
             "Area %s status is: %s. Armed is %s",
             self._area.name,
             self._area.human_status,

homeassistant/components/comelit/config_flow.py

@@ -20,7 +20,7 @@ from homeassistant.core import HomeAssistant
 from homeassistant.exceptions import HomeAssistantError
 from homeassistant.helpers import config_validation as cv
 
-from .const import _LOGGER, CONF_VEDO_PIN, DEFAULT_PORT, DEVICE_TYPE_LIST, DOMAIN
+from .const import CONF_VEDO_PIN, DEFAULT_PORT, DEVICE_TYPE_LIST, DOMAIN, LOGGER
 from .utils import async_client_session
 
 DEFAULT_HOST = "192.168.1.252"
@@ -118,7 +118,7 @@ class ComelitConfigFlow(ConfigFlow, domain=DOMAIN):
         except InvalidVedoAuth:
             errors["base"] = "invalid_vedo_auth"
         except Exception:  # noqa: BLE001
-            _LOGGER.exception("Unexpected exception")
+            LOGGER.exception("Unexpected exception")
             errors["base"] = "unknown"
         else:
             return self.async_create_entry(title=info["title"], data=user_input)
@@ -161,7 +161,7 @@ class ComelitConfigFlow(ConfigFlow, domain=DOMAIN):
             except InvalidPin:
                 errors["base"] = "invalid_pin"
             except Exception:  # noqa: BLE001
-                _LOGGER.exception("Unexpected exception")
+                LOGGER.exception("Unexpected exception")
                 errors["base"] = "unknown"
             else:
                 return self.async_update_reload_and_abort(
@@ -214,7 +214,7 @@ class ComelitConfigFlow(ConfigFlow, domain=DOMAIN):
             except InvalidVedoAuth:
                 errors["base"] = "invalid_vedo_auth"
             except Exception:  # noqa: BLE001
-                _LOGGER.exception("Unexpected exception")
+                LOGGER.exception("Unexpected exception")
                 errors["base"] = "unknown"
             else:
                 data_updates = {

homeassistant/components/comelit/const.py

@@ -9,7 +9,7 @@ from aiocomelit.api import (
 )
 from aiocomelit.const import BRIDGE, VEDO
 
-_LOGGER = logging.getLogger(__package__)
+LOGGER = logging.getLogger(__package__)
 
 type ObjectClassType = (
     ComelitSerialBridgeObject | ComelitVedoAreaObject | ComelitVedoZoneObject

homeassistant/components/comelit/coordinator.py

@@ -32,7 +32,7 @@ from homeassistant.exceptions import ConfigEntryAuthFailed
 from homeassistant.helpers import device_registry as dr
 from homeassistant.helpers.update_coordinator import DataUpdateCoordinator, UpdateFailed
 
-from .const import _LOGGER, DOMAIN, SCAN_INTERVAL, ObjectClassType
+from .const import DOMAIN, LOGGER, SCAN_INTERVAL, ObjectClassType
 
 type ComelitConfigEntry = ConfigEntry[ComelitBaseCoordinator]
 
@@ -63,7 +63,7 @@ class ComelitBaseCoordinator(DataUpdateCoordinator[T]):
 
         super().__init__(
             hass=hass,
-            logger=_LOGGER,
+            logger=LOGGER,
             config_entry=entry,
             name=f"{DOMAIN}-{host}-coordinator",
             update_interval=timedelta(seconds=SCAN_INTERVAL),
@@ -103,7 +103,7 @@ class ComelitBaseCoordinator(DataUpdateCoordinator[T]):
     @override
     async def _async_update_data(self) -> T:
         """Update device data."""
-        _LOGGER.debug("Polling Comelit %s host: %s", self._device, self._host)
+        LOGGER.debug("Polling Comelit %s host: %s", self._device, self._host)
         try:
             await self.api.login()
             return await self._async_update_system_data()
@@ -139,7 +139,7 @@ class ComelitBaseCoordinator(DataUpdateCoordinator[T]):
 
         for i in previous_list:
             if i not in current_list:
-                _LOGGER.debug(
+                LOGGER.debug(
                     "Detected change in %s devices: index %s removed",
                     dev_type,
                     i,

homeassistant/components/comelit/utils.py

@@ -22,7 +22,7 @@ from homeassistant.helpers import (
     entity_registry as er,
 )
 
-from .const import _LOGGER, DOMAIN, ObjectClassType
+from .const import DOMAIN, LOGGER, ObjectClassType
 from .coordinator import ComelitBaseCoordinator
 from .entity import ComelitBridgeBaseEntity
 
@@ -63,7 +63,7 @@ async def cleanup_stale_entity(
     for entry in er.async_entries_for_config_entry(entity_reg, config_entry.entry_id):
         if entry.unique_id == entry_unique_id:
             entry_name = entry.name or entry.original_name
-            _LOGGER.info("Removing entity: %s [%s]", entry.entity_id, entry_name)
+            LOGGER.info("Removing entity: %s [%s]", entry.entity_id, entry_name)
             entity_reg.async_remove(entry.entity_id)
             identifiers.append(f"{config_entry.entry_id}-{device.type}-{device.index}")
 
@@ -82,7 +82,7 @@ def _async_remove_state_config_entry_from_devices(
             (DOMAIN, identifier), config_entry.entry_id
         )
         if device:
-            _LOGGER.info(
+            LOGGER.info(
                 "Removing config entry %s from device %s",
                 config_entry.title,
                 device.name,