Add pre-commit CI and ruff in favor of other linters

Author: codingjoe

.github/workflows/ci.yml

@@ -1,33 +1,10 @@
 name: CI
-
 on:
   push:
     branches:
       - main
   pull_request:
-
 jobs:
-
-  lint:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        lint-command:
-          - bandit -r . -x ./tests
-          - black --check --diff .
-          - flake8 .
-          - isort --check-only --diff .
-          - pydocstyle .
-    steps:
-      - uses: actions/checkout@v5
-      - uses: actions/setup-python@v6
-        with:
-          python-version: "3.x"
-          cache: 'pip'
-          cache-dependency-path: 'pyproject.toml'
-      - run: python -m pip install -e .[lint]
-      - run: ${{ matrix.lint-command }}
-
   dist:
     runs-on: ubuntu-latest
     steps:
@@ -43,11 +20,8 @@ jobs:
       - uses: actions/upload-artifact@v5
         with:
           path: dist/*
-
   pytest-os:
     name: PyTest
-    needs:
-      - lint
     strategy:
       matrix:
         os:
@@ -67,11 +41,8 @@ jobs:
         with:
           flags: ${{ matrix.os }}
           token: ${{ secrets.CODECOV_TOKEN }}
-
   pytest-python:
     name: PyTest
-    needs:
-      - lint
     strategy:
       matrix:
         python-version:
@@ -80,7 +51,7 @@ jobs:
           - "3.12"
           - "3.13"
         django-version:
-          - "4.2"  # LTS
+          - "4.2" # LTS
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v5
@@ -95,11 +66,8 @@ jobs:
         with:
           flags: py${{ matrix.python-version }}
           token: ${{ secrets.CODECOV_TOKEN }}
-
   pytest-django:
     name: PyTest
-    needs:
-      - lint
     strategy:
       matrix:
         python-version:
@@ -122,11 +90,8 @@ jobs:
         with:
           flags: dj${{ matrix.django-version }}
           token: ${{ secrets.CODECOV_TOKEN }}
-
   pytest-extras:
     name: PyTest
-    needs:
-      - lint
     strategy:
       matrix:
         extras:
@@ -151,10 +116,9 @@ jobs:
         with:
           flags: ${{ matrix.extras }}
           token: ${{ secrets.CODECOV_TOKEN }}
-
   codeql:
     name: CodeQL
-    needs: [ dist, pytest-os, pytest-python, pytest-django, pytest-extras ]
+    needs: [dist, pytest-os, pytest-python, pytest-django, pytest-extras]
     runs-on: ubuntu-latest
     permissions:
       actions: read
@@ -163,7 +127,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        language: [ python ]
+        language: [python]
     steps:
       - name: Checkout
         uses: actions/checkout@v5

.pre-commit-config.yaml

@@ -0,0 +1,55 @@
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v6.0.0
+    hooks:
+      - id: trailing-whitespace
+      - id: check-merge-conflict
+      - id: check-ast
+      - id: check-json
+      - id: pretty-format-json
+        args: [--autofix]
+      - id: check-toml
+      - id: check-xml
+      - id: check-yaml
+      - id: check-symlinks
+      - id: debug-statements
+      - id: destroyed-symlinks
+      - id: check-executables-have-shebangs
+      - id: end-of-file-fixer
+      - id: no-commit-to-branch
+        args: [--branch, main]
+      - id: detect-private-key
+        exclude: "fizard/app/services/finapi/openapi/access/api.yaml"
+  - repo: https://github.com/asottile/pyupgrade
+    rev: v3.20.0
+    hooks:
+      - id: pyupgrade
+  - repo: https://github.com/adamchainz/django-upgrade
+    rev: 1.29.0
+    hooks:
+      - id: django-upgrade
+  - repo: https://github.com/hukkin/mdformat
+    rev: 0.7.22
+    hooks:
+      - id: mdformat
+        additional_dependencies:
+          - mdformat-ruff
+          - mdformat-deflist
+          - mdformat-footnote
+          - mdformat-gfm
+          - mdformat-gfm-alerts
+          - mdformat-tables
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.13.3
+    hooks:
+      - id: ruff-check
+        args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format
+  - repo: https://github.com/google/yamlfmt
+    rev: v0.19.0
+    hooks:
+      - id: yamlfmt
+ci:
+  autoupdate_schedule: weekly
+  skip:
+    - no-commit-to-branch

pyproject.toml

@@ -43,13 +43,6 @@ test = [
   "pytest-django",
   "redis",
 ]
-lint = [
-  "bandit==1.8.6",
-  "black==25.9.0",
-  "flake8==7.3.0",
-  "isort==6.1.0",
-  "pydocstyle[toml]==6.3.0",
-]
 dramatiq = [
   "django-dramatiq",
 ]
@@ -102,3 +95,49 @@ skip = ["pictures/_version.py"]
 
 [tool.pydocstyle]
 add_ignore = "D1"
+
+[tool.ruff]
+src = ["flit_gettext", "tests"]
+line-length = 88
+indent-width = 4
+
+[tool.ruff.format]
+quote-style = "double"
+indent-style = "space"
+skip-magic-trailing-comma = false
+line-ending = "auto"
+preview = true
+
+[tool.ruff.lint]
+select = [
+    "D", # pydocstyle
+    "EXE", # flake8-executable
+    "F", # pyflakes
+    "I", # isort
+    "PT", # flake8-pytest-style
+    "RET", # flake8-return
+    "S", # flake8-bandit
+    "SIM", # flake8-simplify
+    "UP", # pyupgrade
+    "W", # pycodestyle warnings
+]
+
+ignore = ["D1"]
+
+[tool.ruff.lint.per-file-ignores]
+"*/test*.py" = ["S101", "E501", "PT011"]
+"*/migrations/*.py" = ["E501"]
+
+[tool.ruff.lint.flake8-pytest-style]
+fixture-parentheses = false
+mark-parentheses = false
+
+[tool.ruff.lint.isort]
+combine-as-imports = true
+split-on-trailing-comma = true
+section-order = ["future", "standard-library", "third-party", "first-party", "local-folder"]
+force-wrap-aliases = true
+known-first-party = ["pictures", "tests"]
+
+[tool.ruff.lint.pydocstyle]
+convention = "pep257"