Skip to content

ci: add GitHub Actions workflow and an nginx deploy example - #3

Merged
ghosteau merged 1 commit into
devfrom
ci
Sep 3, 2026
Merged

ci: add GitHub Actions workflow and an nginx deploy example#3
ghosteau merged 1 commit into
devfrom
ci

Conversation

@ghosteau

@ghosteau ghosteau commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Runs pytest from backend/ (pyproject sets testpaths relative to it and the suite imports app as a top-level package), and oxlint plus the production build for the frontend -- npm run build is 'tsc -b && vite build', so it covers type errors and the bundle the image ships. Python and Node versions match the two stages of the Dockerfile.

torch is installed from PyTorch's CPU index for the same reason the image does it: default PyPI resolves to the CUDA build and drags ~2.5 GB of NVIDIA wheels onto a runner that cannot use them.

The only deploy example was a Caddyfile; this adds nginx for hosts already running it. It carries proxy_read_timeout 300s -- nginx defaults to 60s, which would cut off a first-time model download that the app allows 300s to finish and report it as a confusing 504.

Runs pytest from backend/ (pyproject sets testpaths relative to it and the
suite imports app as a top-level package), and oxlint plus the production
build for the frontend -- npm run build is 'tsc -b && vite build', so it
covers type errors and the bundle the image ships. Python and Node versions
match the two stages of the Dockerfile.

torch is installed from PyTorch's CPU index for the same reason the image
does it: default PyPI resolves to the CUDA build and drags ~2.5 GB of NVIDIA
wheels onto a runner that cannot use them.

The only deploy example was a Caddyfile; this adds nginx for hosts already
running it. It carries proxy_read_timeout 300s -- nginx defaults to 60s,
which would cut off a first-time model download that the app allows 300s to
finish and report it as a confusing 504.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@ghosteau ghosteau added the enhancement New feature or request label Sep 3, 2026
@ghosteau
ghosteau changed the base branch from master to dev September 3, 2026 02:45
@ghosteau
ghosteau merged commit 28a88f8 into dev Sep 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant