Update linters/formatters/precommit to match other projects (#750)

.github/workflows/linting.yaml

@@ -54,31 +54,10 @@ jobs:
           poetry config virtualenvs.in-project true
       - name: 🏗 Install Python dependencies
         run: poetry install --no-interaction
-      - name: 🚀 Run Ruff
-        run: poetry run ruff check .
-
-  black:
-    name: black
-    runs-on: ubuntu-latest
-    steps:
-      - name: ⤵️ Check out code from GitHub
-        uses: actions/[email protected]
-      - name: 🏗 Set up Poetry
-        run: pipx install poetry
-      - name: 🏗 Set up Python ${{ env.DEFAULT_PYTHON }}
-        id: python
-        uses: actions/[email protected]
-        with:
-          python-version: ${{ env.DEFAULT_PYTHON }}
-          cache: "poetry"
-      - name: 🏗 Install workflow dependencies
-        run: |
-          poetry config virtualenvs.create true
-          poetry config virtualenvs.in-project true
-      - name: 🏗 Install Python dependencies
-        run: poetry install --no-interaction
-      - name: 🚀 Run black on docs
-        run: poetry run blacken-docs
+      - name: 🚀 Run ruff linter
+        run: poetry run ruff check --output-format=github .
+      - name: 🚀 Run ruff formatter
+        run: poetry run ruff format --check .
 
   pre-commit-hooks:
     name: pre-commit-hooks
@@ -116,10 +95,6 @@ jobs:
         run: poetry run pre-commit run check-symlinks --all-files
       - name: 🚀 Check TOML files
         run: poetry run pre-commit run check-toml --all-files
-      - name: 🚀 Check XML files
-        run: poetry run pre-commit run check-xml --all-files
-      - name: 🚀 Check YAML files
-        run: poetry run pre-commit run check-yaml --all-files
       - name: 🚀 Check YAML files
         run: poetry run pre-commit run check-yaml --all-files
       - name: 🚀 Detect Private Keys

.pre-commit-config.yaml

@@ -2,25 +2,20 @@
 repos:
   - repo: local
     hooks:
-      - id: ruff
-        name: 🐶 Ruff
+      - id: ruff-check
+        name: 🐶 Ruff Linter
         language: system
         types: [python]
         entry: poetry run ruff check --fix
         require_serial: true
         stages: [commit, push, manual]
-      - id: black
-        name: ☕️ Format using black
+      - id: ruff-format
+        name: 🐶 Ruff Formatter
         language: system
         types: [python]
-        entry: poetry run black
-        require_serial: true
-      - id: blacken-docs
-        name: ☕️ Format documentation examples using black
-        language: system
-        files: '\.(rst|md|markdown|py|tex)$'
-        entry: poetry run blacken-docs
+        entry: poetry run ruff format
         require_serial: true
+        stages: [commit, push, manual]
       - id: check-ast
         name: 🐍 Check Python AST
         language: system

pyproject.toml

@@ -122,13 +122,16 @@ asyncio_mode = "auto"
 
 [tool.ruff.lint]
 ignore = [
-  "ANN101",  # Self... explanatory
-  "ANN102",  # cls... just as annoying as ANN101
   "ANN401",  # Opinioated warning on disallowing dynamically typed expressions
   "D203",    # Conflicts with other rules
   "D213",    # Conflicts with other rules
   "D417",    # False positives in some occasions
   "PLR2004", # Just annoying, not really useful
+
+
+  # Conflicts with the Ruff formatter
+  "COM812",
+  "ISC001",
 ]
 select = ["ALL"]
 

src/demetriek/__init__.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 from .cloud import LaMetricCloud
 from .const import (
     AlarmSound,
@@ -52,8 +53,8 @@
     "DisplayType",
     "Goal",
     "GoalData",
-    "LaMetricCloud",
     "LaMetricAuthenticationError",
+    "LaMetricCloud",
     "LaMetricConnectionError",
     "LaMetricConnectionTimeoutError",
     "LaMetricDevice",

src/demetriek/cloud.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 from __future__ import annotations
 
 import asyncio

src/demetriek/device.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 from __future__ import annotations
 
 import asyncio

src/demetriek/models.py

@@ -1,4 +1,5 @@
 """Models for LaMetric."""
+
 from __future__ import annotations
 
 from datetime import datetime

tests/__init__.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 from pathlib import Path
 
 

tests/test_apps.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import aiohttp
 from aresponses import ResponsesMockServer

tests/test_audio.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import aiohttp
 from aresponses import Response, ResponsesMockServer

tests/test_bluetooth.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import aiohttp
 from aresponses import Response, ResponsesMockServer

tests/test_cloud.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import asyncio
 from datetime import datetime, timezone

tests/test_device.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 from ipaddress import IPv4Address
 

tests/test_display.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import aiohttp
 from aresponses import Response, ResponsesMockServer

tests/test_lametric.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 import asyncio
 

tests/test_wifi.py

@@ -1,4 +1,5 @@
 """Asynchronous Python client for LaMetric TIME devices."""
+
 # pylint: disable=protected-access
 from ipaddress import IPv4Address