Skip to content

ci: add astro check + build smoke as PR gate#21

Merged
Exploitacious merged 1 commit intomainfrom
feat/smoke-tests-major-bumps
May 1, 2026
Merged

ci: add astro check + build smoke as PR gate#21
Exploitacious merged 1 commit intomainfrom
feat/smoke-tests-major-bumps

Conversation

@Exploitacious
Copy link
Copy Markdown
Owner

Phase 1 of Lever 3 from the dependabot-noise audit.

What

New .github/workflows/ci.yml runs npm ci + astro check + npm run build on every PR and push to main.

Why

Repo had a deploy.yml on push to main but no PR-time CI. With Dependabot now opening grouped weekly bumps, every PR needs a real gate. Catches:

  • Astro major-version build regressions (the deferred 5→6 PR is the test case)
  • TS errors introduced by @types/* bumps
  • Tailwind config drift on @tailwindcss/vite bumps
  • Anything npm install can resolve but astro build can't

Repo had a deploy.yml on push to main but no PR-time CI. With
dependabot now opening grouped weekly bumps, every PR needs a real
gate. The new ci.yml runs npm ci + astro check (TS + content
collections) + npm run build on every PR and push.

Catches:

- Astro major-version build regressions (the deferred 5->6 PR is
  the test case).
- TS errors introduced by @types/* bumps.
- Tailwind config drift on @tailwindcss/vite bumps.
- Anything that npm install can resolve but astro build can't.
@Exploitacious Exploitacious merged commit bb3230c into main May 1, 2026
@Exploitacious Exploitacious deleted the feat/smoke-tests-major-bumps branch May 1, 2026 22:19
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