From ef1e6312c820a6ec1cfb30090a7a4b5eb103b239 Mon Sep 17 00:00:00 2001
From: Jakob Schlyter <jakob@kirei.se>
Date: Fri, 3 May 2024 18:45:11 +0200
Subject: [PATCH] use ruff

---
 .github/workflows/ruff.yml               | 10 +++++++++
 .github/workflows/test.yml               | 27 ------------------------
 custom_components/chargeamps/__init__.py | 12 +++++------
 custom_components/chargeamps/light.py    |  4 +---
 custom_components/chargeamps/sensor.py   |  4 +---
 custom_components/chargeamps/switch.py   |  4 +---
 pyproject.toml                           | 22 +++++++++++++++++++
 7 files changed, 41 insertions(+), 42 deletions(-)
 create mode 100644 .github/workflows/ruff.yml
 delete mode 100644 .github/workflows/test.yml
 create mode 100644 pyproject.toml

diff --git a/.github/workflows/ruff.yml b/.github/workflows/ruff.yml
new file mode 100644
index 0000000..3ab0239
--- /dev/null
+++ b/.github/workflows/ruff.yml
@@ -0,0 +1,10 @@
+name: Ruff
+
+on: [ push, pull_request ]
+
+jobs:
+  ruff:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v5
+      - uses: chartboost/ruff-action@v1
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
deleted file mode 100644
index 6d5b3fa..0000000
--- a/.github/workflows/test.yml
+++ /dev/null
@@ -1,27 +0,0 @@
-name: Tests
-
-on: [push, pull_request]
-
-jobs:
-  test:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        python-version:
-          - "3.12"
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v5
-        with:
-          python-version: ${{ matrix.python-version }}
-      - name: Build venv
-        run: python -m venv .venv
-      - name: Install dependencies
-        run: .venv/bin/pip3 install isort black pylama
-      - name: isort
-        run: .venv/bin/isort --check custom_components/chargeamps
-      - name: black
-        run: .venv/bin/black --check custom_components/chargeamps
-      - name: pylama
-        run: .venv/bin/pylama custom_components/chargeamps
diff --git a/custom_components/chargeamps/__init__.py b/custom_components/chargeamps/__init__.py
index 0ba01c1..098ea7c 100644
--- a/custom_components/chargeamps/__init__.py
+++ b/custom_components/chargeamps/__init__.py
@@ -309,9 +309,9 @@ async def _update_data(self, charge_point_id, force: bool = False):
                         charge_point_id, connector_status.connector_id
                     )
                 )
-                self.hass.data[DOMAIN_DATA]["connector_settings"][
-                    key
-                ] = connector_settings
+                self.hass.data[DOMAIN_DATA]["connector_settings"][key] = (
+                    connector_settings
+                )
             total_energy = sum(
                 [
                     v.total_consumption_kwh
@@ -327,9 +327,9 @@ async def _update_data(self, charge_point_id, force: bool = False):
                 round(total_energy, 2)
             )
             settings = await self.client.get_chargepoint_settings(charge_point_id)
-            self.hass.data[DOMAIN_DATA]["chargepoint_settings"][
-                charge_point_id
-            ] = settings
+            self.hass.data[DOMAIN_DATA]["chargepoint_settings"][charge_point_id] = (
+                settings
+            )
         except Exception as error:  # pylint: disable=broad-except
             _LOGGER.error("Could not update data - %s", error)
 
diff --git a/custom_components/chargeamps/light.py b/custom_components/chargeamps/light.py
index 4c53bac..23e5631 100644
--- a/custom_components/chargeamps/light.py
+++ b/custom_components/chargeamps/light.py
@@ -14,9 +14,7 @@
 _LOGGER = logging.getLogger(__name__)
 
 
-async def async_setup_platform(
-    hass, config, async_add_entities, discovery_info=None
-):  # pylint: disable=unused-argument
+async def async_setup_platform(hass, config, async_add_entities, discovery_info=None):  # pylint: disable=unused-argument
     """Setup light platform."""
     lights = []
     handler = hass.data[DOMAIN_DATA]["handler"]
diff --git a/custom_components/chargeamps/sensor.py b/custom_components/chargeamps/sensor.py
index 642b6b9..dfcdc1b 100644
--- a/custom_components/chargeamps/sensor.py
+++ b/custom_components/chargeamps/sensor.py
@@ -15,9 +15,7 @@
 _LOGGER = logging.getLogger(__name__)
 
 
-async def async_setup_platform(
-    hass, config, async_add_entities, discovery_info=None
-):  # pylint: disable=unused-argument
+async def async_setup_platform(hass, config, async_add_entities, discovery_info=None):  # pylint: disable=unused-argument
     """Setup sensor platform."""
     sensors = []
     handler = hass.data[DOMAIN_DATA]["handler"]
diff --git a/custom_components/chargeamps/switch.py b/custom_components/chargeamps/switch.py
index 4d12ed8..cfbf237 100644
--- a/custom_components/chargeamps/switch.py
+++ b/custom_components/chargeamps/switch.py
@@ -10,9 +10,7 @@
 _LOGGER = logging.getLogger(__name__)
 
 
-async def async_setup_platform(
-    hass, config, async_add_entities, discovery_info=None
-):  # pylint: disable=unused-argument
+async def async_setup_platform(hass, config, async_add_entities, discovery_info=None):  # pylint: disable=unused-argument
     """Setup switch platform."""
     switches = []
     handler = hass.data[DOMAIN_DATA]["handler"]
diff --git a/pyproject.toml b/pyproject.toml
new file mode 100644
index 0000000..65fa9f8
--- /dev/null
+++ b/pyproject.toml
@@ -0,0 +1,22 @@
+[tool.isort]
+profile = "black"
+
+[tool.ruff]
+line-length = 132
+
+[tool.ruff.lint]
+select = [
+    # pycodestyle
+    "E",
+    # Pyflakes
+    "F",
+    # pyupgrade
+    "UP",
+    # flake8-bugbear
+    "B",
+    # flake8-simplify
+    "SIM",
+    # isort
+    "I",
+]
+ignore = ["I001"]