[pre-commit.ci] pre-commit autoupdate (#972)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v5.0.0](https://github.com/pre-commit/pre-commit-hooks/compare/v4.5.0...v5.0.0)
- [github.com/astral-sh/ruff-pre-commit: v0.3.5 → v0.8.4](https://github.com/astral-sh/ruff-pre-commit/compare/v0.3.5...v0.8.4)
- [github.com/psf/black: 24.3.0 → 24.10.0](https://github.com/psf/black/compare/24.3.0...24.10.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix issues

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Bas Nijholt <bas@nijho.lt>

.github/update-strings.py

@@ -22,7 +22,7 @@ data = {}
 data_description = {}
 for k, _, typ in const.VALIDATION_TUPLES:
     desc = const.DOCS[k]
-    if len(desc) > 40 and typ != bool and typ != cv.entity_ids:
+    if len(desc) > 40 and typ not in (bool, cv.entity_ids):
         data[k] = k
         data_description[k] = desc
     else:

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.5.0
+    rev: v5.0.0
     hooks:
       - id: check-added-large-files
       - id: trailing-whitespace
@@ -8,11 +8,11 @@ repos:
       - id: mixed-line-ending
         args: ["--fix=lf"]
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.3.5
+    rev: v0.8.4
     hooks:
       - id: ruff
         args: ["--fix"]
   - repo: https://github.com/psf/black
-    rev: 24.3.0
+    rev: 24.10.0
     hooks:
       - id: black

custom_components/adaptive_lighting/adaptation_utils.py

@@ -69,8 +69,8 @@ def _split_service_call_data(service_data: ServiceData) -> list[ServiceData]:
     if service_datas and (transition := service_data.get(ATTR_TRANSITION)) is not None:
         transition /= len(service_datas)
 
-        for service_data in service_datas:
-            service_data[ATTR_TRANSITION] = transition
+        for _service_data in service_datas:
+            _service_data[ATTR_TRANSITION] = transition
 
     return service_datas
 

custom_components/adaptive_lighting/color_and_brightness.py

@@ -64,12 +64,10 @@ class SunEvents:
         ) + self.sunrise_offset
         if self.min_sunrise_time is not None:
             min_sunrise = self._replace_time(dt, self.min_sunrise_time)
-            if min_sunrise > sunrise:
-                sunrise = min_sunrise
+            sunrise = max(min_sunrise, sunrise)
         if self.max_sunrise_time is not None:
             max_sunrise = self._replace_time(dt, self.max_sunrise_time)
-            if max_sunrise < sunrise:
-                sunrise = max_sunrise
+            sunrise = min(max_sunrise, sunrise)
         return sunrise
 
     def sunset(self, dt: datetime.date) -> datetime.datetime:
@@ -81,12 +79,10 @@ class SunEvents:
         ) + self.sunset_offset
         if self.min_sunset_time is not None:
             min_sunset = self._replace_time(dt, self.min_sunset_time)
-            if min_sunset > sunset:
-                sunset = min_sunset
+            sunset = max(min_sunset, sunset)
         if self.max_sunset_time is not None:
             max_sunset = self._replace_time(dt, self.max_sunset_time)
-            if max_sunset < sunset:
-                sunset = max_sunset
+            sunset = min(max_sunset, sunset)
         return sunset
 
     def _replace_time(

custom_components/adaptive_lighting/switch.py

@@ -2552,7 +2552,7 @@ class AdaptiveLightingManager:
             and id_off_to_on == turn_on_event.context.id
         )
 
-    async def just_turned_off(  # noqa: PLR0911, PLR0912
+    async def just_turned_off(  # noqa: PLR0911
         self,
         entity_id: str,
     ) -> bool:

tests/test_switch.py

@@ -2131,11 +2131,8 @@ async def test_light_group(
         assert events[1].context.id == "testing"
         e1 = events[2].data["service_data"][ATTR_ENTITY_ID]
         e2 = events[3].data["service_data"][ATTR_ENTITY_ID]
-        assert (
-            e1 == "light.light_4"
-            and e2 == "light.light_5"
-            or e1 == "light.light_5"
-            and e2 == "light.light_4"
+        assert (e1 == "light.light_4" and e2 == "light.light_5") or (
+            e1 == "light.light_5" and e2 == "light.light_4"
         )
         assert ":lght:" in events[2].context.id
         assert ":lght:" in events[3].context.id

webapp/color_and_brightness.py

@@ -64,12 +64,10 @@ class SunEvents:
         ) + self.sunrise_offset
         if self.min_sunrise_time is not None:
             min_sunrise = self._replace_time(dt, self.min_sunrise_time)
-            if min_sunrise > sunrise:
-                sunrise = min_sunrise
+            sunrise = max(min_sunrise, sunrise)
         if self.max_sunrise_time is not None:
             max_sunrise = self._replace_time(dt, self.max_sunrise_time)
-            if max_sunrise < sunrise:
-                sunrise = max_sunrise
+            sunrise = min(max_sunrise, sunrise)
         return sunrise
 
     def sunset(self, dt: datetime.date) -> datetime.datetime:
@@ -81,12 +79,10 @@ class SunEvents:
         ) + self.sunset_offset
         if self.min_sunset_time is not None:
             min_sunset = self._replace_time(dt, self.min_sunset_time)
-            if min_sunset > sunset:
-                sunset = min_sunset
+            sunset = max(min_sunset, sunset)
         if self.max_sunset_time is not None:
             max_sunset = self._replace_time(dt, self.max_sunset_time)
-            if max_sunset < sunset:
-                sunset = max_sunset
+            sunset = min(max_sunset, sunset)
         return sunset
 
     def _replace_time(