updated linting; fixed various linter issues

Author: rsnodgrass

.github/FUNDING.yml

@@ -0,0 +1,3 @@
+github: rsnodgrass
+patreon: rsnodgrass
+custom: ['https://buymeacoffee.com/DYks67r','https://www.paypal.com/cgi-bin/webscr?cmd=_donations&business=WREP29UDAMB6G']

.gitignore

@@ -8,7 +8,7 @@
 .LSOverride
 
 # Icon must end with two \r
-Icon
+Icon
 
 # Thumbnails
 ._*

.pre-commit-config.yaml

@@ -9,16 +9,8 @@ repos:
       - id: trailing-whitespace
       - id: end-of-file-fixer
 
-  # isort - sort import statements
-#  - repo: https://github.com/pycqa/isort
-#    rev: 5.13.2
-#    hooks:
-#      - id: isort
-#        files: \.(py)$
-#        args: [--settings-path=pyproject.toml]  # ["--profile", "black" ]
-
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.9.2
+    rev: v0.11.10
     hooks:
       - id: ruff
         args: [ --fix ]  # run linter
@@ -34,6 +26,6 @@ repos:
         args: [--py311-plus]    # NOTE: SET TO HOME ASSISTANT VERSION
 
   - repo: https://github.com/dosisod/refurb
-    rev: v2.0.0
+    rev: v2.1.0
     hooks:
       - id: refurb              # NOTE: Update pyproject.toml to match HA Python version

custom_components/xantech/media_player.py

@@ -8,7 +8,7 @@
 from homeassistant.components.media_player import (
     PLATFORM_SCHEMA,
     MediaPlayerEntity,
-    MediaPlayerEntityFeature
+    MediaPlayerEntityFeature,
 )
 from homeassistant.const import (
     ATTR_ENTITY_ID,
@@ -32,8 +32,8 @@
     DOMAIN,
     SERVICE_RESTORE,
     SERVICE_SNAPSHOT,
-#    SERVICE_JOIN,
-#    SERVICE_UNJOIN,
+    #    SERVICE_JOIN,
+    #    SERVICE_UNJOIN,
 )
 
 LOG = logging.getLogger(__name__)
@@ -49,7 +49,7 @@
     | MediaPlayerEntityFeature.VOLUME_SET
     | MediaPlayerEntityFeature.VOLUME_STEP
     | MediaPlayerEntityFeature.TURN_ON
-    | MediaPlayerEntityFeature.TURN_OFF    
+    | MediaPlayerEntityFeature.TURN_OFF
     | MediaPlayerEntityFeature.SELECT_SOURCE
 )
 
@@ -218,7 +218,6 @@ def __init__(self, namespace, name, amp, sources, zone_players):
     async def async_update(self):
         """Retrieve the latest state from the amp."""
         LOG.debug('async_update() is empty')
-        return
 
     @property
     def unique_id(self):
@@ -360,8 +359,7 @@ def state(self):
         power = self._status.get('power')
         if power is not None and power is True:
             return STATE_ON
-        else:
-            return STATE_OFF
+        return STATE_OFF
 
     @property
     def volume_level(self):

custom_components/xantech/services.yaml

@@ -27,4 +27,4 @@ restore:
 #   fields:
 #     entities:
 #       description: Entity ids of the zone players to unjoin.
-#       example: "media_player.xantech8_deck"
+#       example: "media_player.xantech8_deck"

pyproject.toml

@@ -13,6 +13,15 @@ lint.ignore = [
 [tool.ruff.format]
 quote-style = "single" # Use a single quote instead of double
 
+# import sorting settings (replaces isort)
+[tool.ruff.lint.isort]
+force-single-line = false
+force-sort-within-sections = true
+known-first-party = ["custom_components"]
+section-order = ["future", "standard-library", "third-party", "first-party", "local-folder"]
+combine-as-imports = true
+force-to-top = ["logging"]
+
 [tool.pycln]
 all = true