ci: workflow improvements and repo hygiene (#18)

erichutchins · web-flow · commit 6bd8eb36e352 · 2026-03-09T22:39:21.000-04:00
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
@@ -1,16 +1,23 @@
-name: ci
+name: Docs
+
 on:
   push:
     branches:
-      - master
       - main
+    paths:
+      - "docs/**"
+      - "mkdocs.yml"
+      - "polars_iptools/**"
+      - ".github/workflows/docs.yml"
+
 permissions:
   contents: write
+
 jobs:
   deploy:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
       - name: Configure Git Credentials
         run: |
           git config user.name github-actions[bot]
@@ -20,7 +27,7 @@ jobs:
         with:
           python-version: "3.12"
       - run: echo "cache_id=$(date --utc '+%V')" >> $GITHUB_ENV
-      - uses: actions/cache@v4
+      - uses: actions/cache@v6
         with:
           key: mkdocs-material-${{ env.cache_id }}
           path: .cache
diff --git a/.github/workflows/publish_to_pypi.yml b/.github/workflows/publish_to_pypi.yml
@@ -1,12 +1,28 @@
-name: CI
+name: Build & Release
 
 on:
   push:
     branches:
       - main
     tags:
       - "*" # Trigger on tag push for release
+    paths:
+      - "src/**"
+      - "Cargo.toml"
+      - "Cargo.lock"
+      - "pyproject.toml"
+      - "polars_iptools/**"
+      - "tests/**"
+      - ".github/workflows/publish_to_pypi.yml"
   pull_request:
+    paths:
+      - "src/**"
+      - "Cargo.toml"
+      - "Cargo.lock"
+      - "pyproject.toml"
+      - "polars_iptools/**"
+      - "tests/**"
+      - ".github/workflows/publish_to_pypi.yml"
   workflow_dispatch: # Trigger for manual release run
 
 concurrency:
@@ -16,9 +32,10 @@ concurrency:
 permissions:
   contents: read # Default read-only permissions
 
-# Treat all Rust warnings as errors; skip debug symbols to reduce compile time/memory
 env:
   RUSTFLAGS: "-Dwarnings -C debuginfo=0"
+  CARGO_INCREMENTAL: 0
+  PYTHONUTF8: 1
 
 jobs:
   # ------------------------------------------------------------------
@@ -38,7 +55,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - name: Set up Rust stable toolchain
         uses: dtolnay/rust-toolchain@stable
@@ -79,7 +96,7 @@ jobs:
         os: [ubuntu, macos, windows]
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
       - name: install rust stable
         uses: dtolnay/rust-toolchain@stable
@@ -110,7 +127,7 @@ jobs:
   # Build Wheels
   # ------------------------------------------------------------------
   build-linux:
-    name: Build Wheels (${{ matrix.platform.runner }}, ${{ matrix.platform.target }})
+    name: Build Wheels (linux, ${{ matrix.platform.target }})
     if: startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch'
     needs: [test-python, test-os]
     runs-on: ${{ matrix.platform.runner }}
@@ -127,9 +144,9 @@ jobs:
             target: armv7
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v6
         with:
           python-version: 3.x
 
@@ -141,14 +158,20 @@ jobs:
           sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
           manylinux: auto
 
+      - name: Smoke test wheel
+        if: matrix.platform.target == 'x86_64'
+        run: |
+          pip install --force-reinstall --verbose dist/*.whl
+          python -c 'import polars_iptools'
+
       - name: Upload wheels
         uses: actions/upload-artifact@v4
         with:
           name: wheels-linux-${{ matrix.platform.target }}
           path: dist
 
   build-musllinux:
-    name: Build Wheels (${{ matrix.platform.runner }}, ${{ matrix.platform.target }})
+    name: Build Wheels (musllinux, ${{ matrix.platform.target }})
     if: startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch'
     needs: [test-python, test-os]
     runs-on: ${{ matrix.platform.runner }}
@@ -165,9 +188,9 @@ jobs:
             target: armv7
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v6
         with:
           python-version: 3.x
 
@@ -186,7 +209,7 @@ jobs:
           path: dist
 
   build-windows:
-    name: Build Wheels (${{ matrix.platform.runner }}, ${{ matrix.platform.target }})
+    name: Build Wheels (windows, ${{ matrix.platform.target }})
     if: startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch'
     needs: [test-python, test-os]
     runs-on: ${{ matrix.platform.runner }}
@@ -201,9 +224,9 @@ jobs:
             target: x86
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v6
         with:
           python-version: 3.x
           architecture: ${{ matrix.platform.target }}
@@ -215,14 +238,20 @@ jobs:
           args: --release --out dist
           sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
 
+      - name: Smoke test wheel
+        if: matrix.platform.target == 'x64'
+        run: |
+          pip install --force-reinstall --verbose dist/*.whl
+          python -c 'import polars_iptools'
+
       - name: Upload wheels
         uses: actions/upload-artifact@v4
         with:
           name: wheels-windows-${{ matrix.platform.target }}
           path: dist
 
   build-macos:
-    name: Build Wheels (${{ matrix.platform.runner }}, ${{ matrix.platform.target }})
+    name: Build Wheels (macos, ${{ matrix.platform.target }})
     if: startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch'
     needs: [test-python, test-os]
     runs-on: ${{ matrix.platform.runner }}
@@ -231,15 +260,15 @@ jobs:
       fail-fast: false
       matrix:
         platform:
-          - runner: macos-14
+          - runner: macos-15-intel
             target: x86_64
           - runner: macos-15
             target: aarch64
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
 
-      - uses: actions/setup-python@v5
+      - uses: actions/setup-python@v6
         with:
           python-version: 3.x
 
@@ -250,6 +279,11 @@ jobs:
           args: --release --out dist
           sccache: ${{ !startsWith(github.ref, 'refs/tags/') }}
 
+      - name: Smoke test wheel
+        run: |
+          pip install --force-reinstall --verbose dist/*.whl
+          python -c 'import polars_iptools'
+
       - name: Upload wheels
         uses: actions/upload-artifact@v4
         with:
@@ -264,7 +298,7 @@ jobs:
     if: startsWith(github.ref, 'refs/tags/') || github.event_name == 'workflow_dispatch'
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
       - name: Build sdist
         uses: PyO3/maturin-action@v1
         with:
@@ -296,15 +330,15 @@ jobs:
       attestations: write
     steps:
       - uses: actions/download-artifact@v4
+        with:
+          path: dist
+          merge-multiple: true
       - name: Generate artifact attestation
         uses: actions/attest-build-provenance@v2
         with:
-          subject-path: "wheels-*/*"
+          subject-path: "dist/*"
       - name: Publish to PyPI
         if: ${{ startsWith(github.ref, 'refs/tags/') }}
-        uses: PyO3/maturin-action@v1
-        env:
-          MATURIN_PYPI_TOKEN: ${{ secrets.PYPI_API_TOKEN }}
+        uses: pypa/gh-action-pypi-publish@release/v1
         with:
-          command: upload
-          args: --non-interactive --skip-existing wheels-*/*
+          skip-existing: true
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,7 +1,7 @@
 repos:
   - repo: https://github.com/astral-sh/ruff-pre-commit
     # Ruff version.
-    rev: v0.7.2
+    rev: v0.15.1
     hooks:
       # Run the linter.
       - id: ruff
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "polars-iptools"
-version = "0.2.0"
+version = "0.2.1"
 edition = "2021"
 license = "MIT"
 repository = "https://github.com/erichutchins/polars_iptools"
diff --git a/Makefile b/Makefile
diff --git a/justfile b/justfile
@@ -33,11 +33,13 @@ install: setup require-maturin
 install-release: setup require-maturin
     unset CONDA_PREFIX && source .venv/bin/activate && maturin develop --uv --release
 
-# Run pre-commit checks
-pre-commit: setup
-    uv run pre-commit install
-    uv run pre-commit run --all-files
-    uv run mypy polars_iptools tests
+# Install prek git hooks
+hooks:
+    prek install
+
+# Run all pre-commit hooks
+lint:
+    prek run --all-files
 
 # Clean up build artifacts
 clean:
@@ -72,8 +74,3 @@ run-release: install-release
 # Test mkdocs locally
 docs-serve:
     uv run --group docs mkdocs serve
-
-# Lint and format python code
-lint:
-    uv run --no-sync ruff format src tests
-    uv run --no-sync ruff check --fix src tests
diff --git a/pyproject.toml b/pyproject.toml
@@ -62,7 +62,6 @@ dev = [
     "hatch>=1.14.0",
     "maturin>=1.0,<2.0",
     "mypy>=1.11.2",
-    "pre-commit>=4.1.0",
     "pytest>=8.3.3",
     "ruff>=0.9.6",
 ]
diff --git a/uv.lock b/uv.lock

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,6 @@ dev = [`
`62`	`62`	`"hatch>=1.14.0",`
`63`	`63`	`"maturin>=1.0,<2.0",`
`64`	`64`	`"mypy>=1.11.2",`
`65`		`- "pre-commit>=4.1.0",`
`66`	`65`	`"pytest>=8.3.3",`
`67`	`66`	`"ruff>=0.9.6",`
`68`	`67`	`]`