Run pre-commit in GitHub Actions (#72)

* make pre-commit pass

* Bump pre-commit repos

* Rename deprecated hook

* Add no-commit-to-branch hook

* Run pre-commit in GitHub Actions

* Lint is not test, extract in a different workflow

Author: serl

.github/workflows/lint.yml

@@ -0,0 +1,41 @@
+name: lint
+
+on:
+  pull_request:
+  push:
+    branches: [main]
+  workflow_call:
+
+permissions:
+  contents: read
+
+env:
+  CARGO_TERM_COLOR: always
+  FORCE_COLOR: "1"
+  PYTHONUNBUFFERED: "1"
+  UV_VERSION: "0.4.x"
+
+jobs:
+  pre-commit:
+    runs-on: ubuntu-24.04
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v5
+        with:
+          enable-cache: true
+          version: ${{ env.UV_VERSION }}
+
+      - uses: actions/cache@v4
+        with:
+          path: ~/.cache/pre-commit/
+          key: pre-commit-1|${{ hashFiles('.pre-commit-config.yaml') }}
+
+      - name: pre-commit
+        run: |
+          SKIP=no-commit-to-branch \
+          uv run --with pre-commit-uv pre-commit run \
+            --all-files \
+            --show-diff-on-failure \
+            --color always

.pre-commit-config.yaml

@@ -1,16 +1,19 @@
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.6.0
+    rev: v5.0.0
     hooks:
-      - id: check-byte-order-marker
+      - id: fix-byte-order-marker
       - id: check-case-conflict
       - id: check-merge-conflict
       - id: check-symlinks
       - id: check-toml
       - id: check-yaml
+        args: ["--unsafe"] # breaks on .mkdocs.yml otherwise
       - id: end-of-file-fixer
       - id: mixed-line-ending
       - id: trailing-whitespace
+      - id: no-commit-to-branch
+        args: ["--branch", "main"]
   - repo: https://github.com/backplane/pre-commit-rust-hooks
     rev: v1.1.0
     hooks: