Skip to content

ci: add GitHub Actions quality pipeline - #4

Merged
bjarkeef merged 3 commits into
mainfrom
feat/add-github-ci
Jun 23, 2026
Merged

ci: add GitHub Actions quality pipeline#4
bjarkeef merged 3 commits into
mainfrom
feat/add-github-ci

Conversation

@bjarkeef

Copy link
Copy Markdown
Owner

Summary

  • Add .github/workflows/ci.yml with quality (validate/lint/typecheck) and build (Next production build, no migrate) jobs on PRs and pushes to main.
  • Add
    pm run typecheck, db:validate, and
    pm run ci scripts.
  • Only run prisma migrate deploy on Vercel (VERCEL=1) or explicit RUN_DB_MIGRATE=1, so GitHub Actions does not need a live database.
  • Fix a few existing lint/type errors so CI can pass on current main (route planner types, stats empty return, badge variant, leaflet icon cast).
  • Document the pipeline in docs/deployment.md.

Test plan

  • Open PR and confirm CI / Lint, typecheck & validate and CI / Next.js build run green
  • Locally:
    pm run ci succeeds
  • Locally:
    pm run build still skips migrate without Vercel env
  • After merge: enable branch protection requiring the CI checks on main (recommended)

Introduce PR/main CI (Prisma validate, ESLint, tsc, Next build without DB migrate). Adjust build migrate gates for Vercel-only deploys and fix existing lint/type blockers so the pipeline is green.
@vercel

vercel Bot commented Jun 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
runnr Ready Ready Preview, Comment Jun 23, 2026 3:30pm

Lockfile was already out of sync with package.json (missing transitive @radix-ui / floating-ui entries), so GitHub Actions npm ci failed. Scripts-only package.json edits did not cause this; refresh lockfile from current dependencies.
postinstall runs prisma generate which loaded prisma.config.ts and hard-required env(DATABASE_URL), breaking npm ci on Actions. Use a placeholder URL when unset and set dummy DB env on the whole CI workflow.
@bjarkeef
bjarkeef merged commit d6eb3eb into main Jun 23, 2026
4 checks passed
@bjarkeef
bjarkeef deleted the feat/add-github-ci branch June 23, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant