.github/workflows/test.yml

@@ -23,7 +23,7 @@ jobs:
         with:
           fetch-depth: 2
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v5.6.0
+        uses: actions/setup-python@v5.4.0
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install dependencies

custom_components/rpi_gpio/binary_sensor.py

@@ -52,20 +52,17 @@ async def async_setup_platform(
 
     sensors = []
     for sensor in config.get(CONF_SENSORS):
-        try:
+        sensors.append(
-            sensors.append(
+            GPIODBinarySensor(
-                GPIODBinarySensor(
+                hub,
-                    hub,
+                sensor[CONF_NAME],
-                    sensor[CONF_NAME],
+                sensor[CONF_PORT],
-                    sensor[CONF_PORT],
+                sensor.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{sensor[CONF_PORT]}_{sensor[CONF_NAME].lower().replace(' ', '_')}",
-                    sensor.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{sensor[CONF_PORT]}_{sensor[CONF_NAME].lower().replace(' ', '_')}",
+                sensor.get(CONF_INVERT_LOGIC),
-                    sensor.get(CONF_INVERT_LOGIC),
+                sensor.get(CONF_PULL_MODE),
-                    sensor.get(CONF_PULL_MODE),
+                sensor.get(CONF_BOUNCETIME)
-                    sensor.get(CONF_BOUNCETIME)
-                )
             )
-        except Exception as e:
+        )
-            _LOGGER.error(f"Failed to add binary sensor {sensor[CONF_NAME]} for port {sensor[CONF_PORT]}: {e}")
 
     async_add_entities(sensors)
 

custom_components/rpi_gpio/cover.py

@@ -71,24 +71,21 @@ async def async_setup_platform(
     invert_relay = config[CONF_INVERT_RELAY]
     covers = []
     for cover in config.get(CONF_COVERS):
-        try:
+        covers.append(
-            covers.append(
+            GPIODCover(
-                GPIODCover(
+                hub,
-                    hub,
+                cover[CONF_NAME],
-                    cover[CONF_NAME],
+                cover.get(CONF_RELAY_PIN),
-                    cover.get(CONF_RELAY_PIN),
+                relay_time,
-                    relay_time,
+                invert_relay,
-                    invert_relay,
+                "AS_IS",
-                    "AS_IS",
+                "PUSH_PULL",
-                    "PUSH_PULL",
+                cover.get(CONF_STATE_PIN),
-                    cover.get(CONF_STATE_PIN),
+                state_pull_mode,
-                    state_pull_mode,
+                invert_state,
-                    invert_state,
+                cover.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{cover.get(CONF_RELAY_PIN)}_{cover[CONF_NAME].lower().replace(' ', '_')}",
-                    cover.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{cover.get(CONF_RELAY_PIN)}_{cover[CONF_NAME].lower().replace(' ', '_')}",
-                )
             )
-        except Exception as e:
+        )
-            _LOGGER.error(f"Failed to add cover {cover[CONF_NAME]} for port {cover.get(CONF_RELAY_PIN)}:{cover.get(CONF_STATE_PIN)}: {e}")
 
     async_add_entities(covers)
 

custom_components/rpi_gpio/manifest.json

@@ -7,5 +7,5 @@
   "iot_class": "local_push",
   "issue_tracker": "https://github.com/thecode/ha-rpi_gpio/issues",
   "requirements": [ "gpiod>=2.2.1" ],
-  "version": "2025.2.1"
+  "version": "2025.2.0"
 }

custom_components/rpi_gpio/switch.py

@@ -56,21 +56,18 @@ async def async_setup_platform(
 
     switches = []
     for switch in config.get(CONF_SWITCHES):
-        try:
+        switches.append(
-            switches.append(
+            GPIODSwitch(
-                GPIODSwitch(
+                hub,
-                    hub,
+                switch[CONF_NAME],
-                    switch[CONF_NAME],
+                switch[CONF_PORT],
-                    switch[CONF_PORT],
+                switch.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{switch[CONF_PORT]}_{switch[CONF_NAME].lower().replace(' ', '_')}",
-                    switch.get(CONF_UNIQUE_ID) or f"{DOMAIN}_{switch[CONF_PORT]}_{switch[CONF_NAME].lower().replace(' ', '_')}",
+                switch.get(CONF_INVERT_LOGIC),
-                    switch.get(CONF_INVERT_LOGIC),
+                switch.get(CONF_PULL_MODE),
-                    switch.get(CONF_PULL_MODE),
+                switch.get(CONF_DRIVE),
-                    switch.get(CONF_DRIVE),
+                switch[CONF_PERSISTENT]
-                    switch[CONF_PERSISTENT]
-                )
             )
-        except Exception as e:
+        )
-            _LOGGER.error(f"Failed to add switch {switch[CONF_NAME]} for port {switch[CONF_PORT]}: {e}")
 
     async_add_entities(switches)
 

requirements_lint.txt

@@ -1,7 +1,7 @@
-black==25.1.0
+black==24.10.0
-flake8==7.3.0
+flake8==7.1.1
-isort==6.0.1
+isort==5.13.2
-mypy==1.17.0
+mypy==1.14.1
-pre-commit==4.2.0
+pre-commit==4.1.0
 pydocstyle==6.3.0
-pylint==3.3.7
+pylint==3.3.3