ci: migrate to uv, add pre-commit hooks, and expand linting rules

  - Switch CI from pip to uv (astral-sh/setup-uv) with built-in caching
  - Add Python 3.11/3.12 matrix and concurrency group to cancel stale runs
  - Add docker-compose build check as separate job
  - Upload coverage report to GitHub Job Summary
  - Add ruff and pyright to dev dependencies in pyproject.toml
  - Migrate build backend from setuptools to hatchling
  - Expand ruff lint rules: bugbear, pyupgrade, simplify, numpy, pytest-style
  - Add .pre-commit-config.yaml with ruff lint and format hooks

Author: ennomuller

.github/workflows/test.yml

@@ -6,34 +6,48 @@ on:
   pull_request:
     branches: [main]
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 jobs:
-  ci:
+  test-and-lint:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.11", "3.12"]
 
     steps:
       - uses: actions/checkout@v4
 
-      - uses: actions/setup-python@v5
-        with:
-          python-version: "3.11"
-
-      - uses: actions/cache@v4
+      - name: Install uv
+        uses: astral-sh/setup-uv@v5
         with:
-          path: ~/.cache/pip
-          key: ${{ runner.os }}-pip-${{ hashFiles('pyproject.toml') }}
-          restore-keys: ${{ runner.os }}-pip-
+          python-version: ${{ matrix.python-version }}
+          enable-cache: true
 
-      - name: Install package and dev dependencies
-        run: pip install -e ".[dev,notebooks]"
+      - name: Install dependencies
+        run: uv sync --all-extras
 
-      - name: Install linting and type-checking tools
-        run: pip install ruff pyright
+      - name: Check formatting (Ruff)
+        run: uv run ruff format --check .
 
-      - name: Ruff lint
-        run: ruff check . --config pyproject.toml
+      - name: Lint (Ruff)
+        run: uv run ruff check --output-format=github .
 
-      - name: Pyright type check
-        run: pyright src/ --warnings
+      - name: Type check (Pyright)
+        run: uv run pyright src/
 
       - name: Pytest with coverage
-        run: pytest --cov=src/heliotrace --cov-report=term-missing
+        run: uv run pytest --cov=src/heliotrace --cov-report=xml --cov-report=term-missing
+
+      - name: Upload coverage to Job Summary
+        if: matrix.python-version == '3.11' # Only run once
+        run: uv run coverage report --format=markdown >> $GITHUB_STEP_SUMMARY
+
+  docker-check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Test Docker Build
+        run: docker compose build

.pre-commit-config.yaml

@@ -0,0 +1,7 @@
+repos:
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.3.0 # replace with current ruff version
+    hooks:
+      - id: ruff
+        args: [ --fix ]
+      - id: ruff-format

pyproject.toml

@@ -14,23 +14,28 @@ dependencies = [
 ]
 
 [project.optional-dependencies]
-# Install with: pip install heliotrace[notebooks]
+dev = [
+    "pytest>=8.0",
+    "pytest-cov",
+    "ruff",
+    "pyright",
+]
 notebooks = [
     "sunpy~=5.1",
     "matplotlib~=3.10",
 ]
-# Install with: pip install heliotrace[dev]
+
+[dependency-groups]
 dev = [
     "pytest>=8.0",
     "pytest-cov",
+    "ruff",
+    "pyright",
 ]
 
 [build-system]
-requires = ["setuptools>=61.0"]
-build-backend = "setuptools.build_meta"
-
-[tool.setuptools.packages.find]
-where = ["src"]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
@@ -42,16 +47,22 @@ addopts   = "-v"
   exclude = [".venv", ".git", "__pycache__"]
 
   [tool.ruff.lint]
-  select = ["E", "F", "W", "I"]
-  ignore = ["E501", "F401"] 
+  select =[
+    "E", "F", "W",  # Standard Flake8
+    "I",            # isort
+    "B",            # flake8-bugbear
+    "UP",           # pyupgrade: modern Python 3.11+ syntax
+    "SIM",          # flake8-simplify: cleaner code logic
+    "NPY",          # NumPy-specific rules: for Astropy/SciPy workflows
+    "PT",           # flake8-pytest-style: keeps tests clean
+  ]
+  ignore = ["E501"] 
 
   [tool.ruff.lint.isort]
   known-first-party = ["heliotrace"]
 
 [tool.pyright]
 pythonVersion = "3.11"
-venvPath = "."
-venv = ".venv"
 typeCheckingMode = "basic"
 reportMissingTypeStubs = false
 reportAttributeAccessIssue = "warning" # CI shouldn't fail on sol.t or array.to()