Improve python tooling, CI and testing (evcc-io#26)

Author: lfloeer

.dockerignore

@@ -1,9 +1,11 @@
-# flyctl launch added from .gitignore
-**/.claude
-**/.vscode
-**/.git
+# Ignore all files
+*
 
-# flyctl launch added from .venv/.gitignore
-# Created by venv; see https://docs.python.org/3/library/venv.html
-.venv/**/*
-fly.toml
+# Except what we really need
+!uv.lock
+!pyproject.toml
+!README.md
+!src/
+
+# Exclude files in the included directories that we don't want
+__pycache__

.flake8

@@ -0,0 +1,39 @@
+name: CI
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v5
+      - name: Install uv
+        uses: astral-sh/setup-uv@v6
+        with:
+          enable-cache: true
+      - name: Format
+        run: uv run autopep8 --exit-code --diff . 
+      - name: Lint
+        run: uv run ruff check
+      - name: Test
+        run: uv run pytest
+
+  docker:
+    name: Docker
+    runs-on: ubuntu-latest
+    needs: test
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v5
+      - name: Build
+        uses: docker/build-push-action@v6
+        with:
+          push: false
+          # To be changed
+          tags: andig/evopt:latest

.github/workflows/ci.yml

@@ -1,2 +1,4 @@
 .claude/
-.vscode/
+.vscode/
+.venv/
+__pycache__/

.gitignore

@@ -0,0 +1 @@
+3.13

.pylintrc

@@ -1,24 +1,27 @@
-FROM python:3.13-slim
+# Install uv
+FROM python:3.13-slim AS builder
+COPY --from=ghcr.io/astral-sh/uv:latest /uv /uvx /bin/
 
+# Change the working directory to the `app` directory
 WORKDIR /app
 
-# Install system dependencies including CBC solver for PuLP
-RUN apt-get update && apt-get install -y \
-    gcc \
-    g++ \
-    coinor-cbc \
-    htop \
-    && rm -rf /var/lib/apt/lists/*
+# Install dependencies
+RUN --mount=type=cache,target=/root/.cache/uv \
+    --mount=type=bind,source=uv.lock,target=uv.lock \
+    --mount=type=bind,source=pyproject.toml,target=pyproject.toml \
+    uv sync --locked --no-install-project --no-editable --no-group dev
+
+# Copy the project into the intermediate image
+ADD . /app
 
-# Copy requirements and install Python dependencies
-COPY requirements.txt .
-RUN pip install --upgrade --no-cache-dir -r requirements.txt
+# Sync the project
+RUN --mount=type=cache,target=/root/.cache/uv \
+    uv sync --locked --no-editable --no-group dev
 
-# Copy application code
-COPY *.py .
+FROM python:3.13-slim
 
-# Expose the port
-EXPOSE 7050
+# Copy the environment, but not the source code
+COPY --from=builder --chown=app:app /app/.venv /app/.venv
 
 # Run the application
-CMD ["gunicorn", "--bind", "0.0.0.0:7050", "--workers", "4", "--max-requests", "32", "app:app"]
+CMD ["/app/.venv/bin/gunicorn", "--bind", "0.0.0.0:7050", "--workers", "4", "--max-requests", "32", "evopt.app:app"]

.python-version

@@ -5,25 +5,32 @@ default: build docker-build
 build::
 	go generate ./...
 
-test::
-	go run example/client.go
-
 install::
-	python3 -m venv .venv && source .venv/bin/activate && pip install --upgrade -r requirements.txt
+	uv sync
 
 upgrade::
-	python3 -m venv .venv && source .venv/bin/activate && pip install --upgrade $$(pip list --outdated | awk 'NR>2 {print $$1}')
+	uv lock --upgrade
+
+lint::
+	uv run autopep8 . --in-place
+	uv run ruff check --fix
+
+test::
+	uv run pytest
 
 run::
-	source .venv/bin/activate && python3 app.py
+	uv run python -m evopt.app
 
-docker: docker-build docker-run
+docker: docker-build docker-push docker-run
 
 docker-build::
-	docker buildx build . --tag $(DOCKER_IMAGE) --push
+	docker buildx build . --tag $(DOCKER_IMAGE)
 
 docker-run::
 	docker run -p 7050:7050 -it $(DOCKER_IMAGE)
 
+docker-push::
+	docker push $(DOCKER_IMAGE)
+
 fly::
 	fly deploy --local-only

Dockerfile

@@ -0,0 +1,41 @@
+[project]
+name = "evopt"
+version = "0.1.0"
+description = "Add your description here"
+readme = "README.md"
+authors = [{ name = "andig", email = "andi@evcc.io" }, { name = "ekke" }]
+requires-python = "~=3.13.0"
+dependencies = [
+    "flask>=3.0.3",
+    "flask-restx>=1.3.0",
+    "gunicorn>=22.0.0",
+    "numpy>=2.3.1",
+    "pulp>=3.2.2",
+    "pyjwt>=2.8.0",
+    "werkzeug>=3.0.3",
+]
+
+[project.scripts]
+evopt = "evopt:main"
+
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[dependency-groups]
+dev = [
+    "autopep8>=2.3.2",
+    "pytest>=8.4.2",
+    "ruff>=0.12.12",
+]
+
+[tool.autopep8]
+max_line_length = 160
+recursive = true
+
+[tool.ruff]
+line-length = 160
+
+[tool.ruff.lint]
+select = ["F", "I", "E", "W", "PL"]
+ignore = ["PLR0912", "PLR0913", "PLR2004", "PLR0917", "PLR6301"]