simple fixes

conf/apps/deep/arbitrary/path/statemachine.py

@@ -4,5 +4,4 @@ logger = logging.getLogger('AppDaemon.Perimeter')
 logger.info('Imported statemachine')
 
 
-class StateMachine:
+class StateMachine: ...
-    ...

conf/apps/deep/arbitrary/rules.py

@@ -5,8 +5,8 @@ from dataclasses import dataclass
 class Rule1:
     state: str
 
+
 @dataclass
 class Rule2:
     value: int
-    other: str = "default" # test changing this
+    other: str = 'default'  # test changing this
-

conf/apps/family/grand_parent.py

@@ -2,5 +2,5 @@ from appdaemon.adapi import ADAPI
 
 
 class GrandParent(ADAPI):
-    def initialize(self):
+    def initialize(self) -> None:
         self.log(f'{self.__class__.__name__} Initialized')

conf/apps/family/parent.py

@@ -2,5 +2,5 @@ from appdaemon.adapi import ADAPI
 
 
 class Parent(ADAPI):
-    def initialize(self):
+    def initialize(self) -> None:
         self.log(f'{self.__class__.__name__} Initialized')

conf/apps/family/sibling.py

@@ -1,7 +1,6 @@
-
 from child import Child
 
 
 class Sibling(Child):
-    def initialize(self):
+    def initialize(self) -> None:
         self.log(f'{self.__class__.__name__} Initialized')

conf/apps/food-repo/src/food/menu.py

@@ -1,5 +1,4 @@
 from dataclasses import dataclass, field
-from typing import List
 
 from .base import FoodItem
 from .eggs import Eggs
@@ -12,7 +11,7 @@ class GreenEggs(Eggs):
 
 @dataclass
 class Menu:
-    dishes: List[type[FoodItem]] = field(init=False)
+    dishes: list[type[FoodItem]] = field(init=False)
 
     def __post_init__(self):
         self.dishes = [GreenEggs, Ham]

conf/apps/food-repo/src/food/utils.py

@@ -13,4 +13,5 @@ class Utensil(Enum):
     POT = auto()
     PAN = auto()
 
+
 GLOBAL_VAR = Utensil.FORK

conf/apps/food-repo/src/restaurant/restaurant.py

@@ -5,7 +5,7 @@ from food.menu import Menu
 class Restaurant(ADAPI):
     menu: Menu
 
-    def initialize(self):
+    def initialize(self) -> None:
         self.log(f'{self.__class__.__name__} initialized')
 
         self.menu = Menu()

conf/apps/globals.py

@@ -1,18 +1,22 @@
+import logging
 from enum import Enum
 
-GLOBAL_VAR = "Hello, World!"
+LOGGER = logging.getLogger('AppDaemon._globals')
 
 
-def global_function():
+GLOBAL_VAR = 'Hello, World!'
-    print('This is a global function.')
+
+
+def global_function() -> None:
+    LOGGER.info('This is a global function.')
 
 
 class GlobalClass:
-    def __init__(self):
+    def __init__(self) -> None:
         self.value = 'This is a global class instance.'
 
-    def display(self):
+    def display(self) -> None:
-        print(self.value)
+        LOGGER.info(self.value)
 
 
 class ModeSelect(Enum):

conf/apps/globals/hal.py

@@ -3,8 +3,9 @@ from typing import Any
 
 logger = logging.getLogger('AppDaemon.Perimeter')
 
+
 class HAL:
-    def __init__(self, *args: Any, **kwargs: Any):
+    def __init__(self, *args: Any, **kwargs: Any) -> None:
         self.args = args
         self.kwargs = kwargs
         logger.info('Logging from HAL')

conf/apps/globals/utils.py

@@ -1,8 +1,9 @@
 CONSTANTS = {
     'A': 1,
     'B': 2,
-    'C': 3
+    'C': 3,
 }
 
+
 def utility_function():
     return 123, 456

conf/apps/simple_app/apps.yaml

@@ -12,17 +12,16 @@ simple_app:
   # dependencies:
   #   - hello_world
 
-
 base_app:
   module: simple
   class: BaseApp
 
-AppA:
+# AppA:
-  module: app_a
+#   module: app_a
-  class: AppA
+#   class: AppA
-  dependencies:
+#   dependencies:
-    - AppB # This is only set to demonstrate forcing it to load after AppB
+#     - AppB # This is only set to demonstrate forcing it to load after AppB
 
-AppB:
+# AppB:
-  module: app_b
+#   module: app_b
-  class: AppB
+#   class: AppB

conf/apps/simple_app/hello.py

@@ -10,14 +10,15 @@ from appdaemon.adapi import ADAPI
 # fake/
 # SimpleApp
 
+
 class HelloWorld(ADAPI):
-    def initialize(self):
+    def initialize(self) -> None:
         self.log(f'{self.__class__.__name__} Initialized')
         self.log('+' * 50)
         # fake
-        self.register_service("my_domain/my_exciting_service", self.my_exciting_cb)
+        self.register_service('my_domain/my_exciting_service', self.my_exciting_cb)
 
     def my_exciting_cb(self, *args: str, my_arg: int = 0, **kwargs: Any) -> Any:
         namespace, domain, service = args
-        self.log(f"Service {domain}/{service} in the {namespace} namepsace called with {kwargs}")
+        self.log(f'Service {domain}/{service} in the {namespace} namepsace called with {kwargs}')
         return 999 + my_arg

conf/apps/simple_app/simple.py

@@ -8,7 +8,7 @@ class SimpleApp(Hass):
         match self.ping():
             case float() as ping:
                 ping = utils.format_timedelta(ping)
-                self.log(f"{self.__class__.__name__} Initialized: {ping}")
+                self.log(f'{self.__class__.__name__} Initialized: {ping}')
             case _:
                 pass
 
@@ -17,5 +17,5 @@ class BaseApp(ADBase):
     def initialize(self) -> None:
         self.adapi = self.get_ad_api()
         self.log = self.adapi.log
-        self.hassapi = self.get_plugin_api("HASS")
+        self.hassapi = self.get_plugin_api('HASS')
-        assert isinstance(self.hassapi, Hass), "HASS API not available"
+        assert isinstance(self.hassapi, Hass), 'HASS API not available'

ruff.toml

@@ -1,14 +1,29 @@
-line-length = 88
+line-length = 100
 target-version = "py312"
 
+[format]
+quote-style = "single"
+indent-style = "space"
+
 [lint]
 exclude = [
     "conf/apps/simple_app/malformed.py"
 ]
 
 select = ["ALL"]
-extend-ignore = ["ANN", "D", "E501", "COM812", "ERA001", "S101", "INP001"]
+extend-ignore = [
+    "ANN",
+    "BLE001",
+    "COM812",
+    "D",
+    "E501",
+    "ERA001",
+    "INP001",
+    "S101",
+]
 
-[format]
+
-quote-style = "single"
+[lint.flake8-quotes]
-indent-style = "space"
+inline-quotes = "single"
+docstring-quotes = "double"
+multiline-quotes = "double"