♻️ Move Ruff linting logic to scripts/ruff

Author: Jamim

.github/workflows/ci.yml

@@ -47,8 +47,6 @@ jobs:
       - name: Install dependencies
         run: pdm install -G lint --frozen-lockfile
       - name: Lint with ruff
-        run: |
-          pdm run ruff check .
-          pdm run ruff format --check .
+        run: pdm run lint-ruff
       - name: Type check with mypy
         run: pdm run mypy

Dockerfile

@@ -10,6 +10,7 @@ RUN emerge -q dev-python/tox
 
 WORKDIR /usr/src/gcm
 COPY pyproject.toml README.md tox.ini ./
+COPY scripts scripts
 COPY src src
 COPY tests tests
 

pyproject.toml

@@ -64,6 +64,9 @@ test = [
     "pytest-env",
 ]
 
+[tool.pdm.scripts]
+lint-ruff = "scripts/ruff"
+
 [tool.ruff]
 line-length = 79
 

scripts/ruff

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+set -ex
+
+ruff check src
+ruff check tests \
+    --config 'lint.isort.known-first-party = ["gcm"]' \
+    --config 'lint.isort.known-local-folder = ["tests"]'
+ruff format --check

tox.ini

@@ -15,9 +15,8 @@ commands = pytest -v
 [testenv:lint-ruff]
 skip_install = true
 deps = ruff
-commands =
-    ruff check .
-    ruff format --check .
+allowlist_externals = scripts/ruff
+commands = scripts/ruff
 
 [testenv:lint-mypy]
 package = editable