ci: run the check job on pull requests (#68) #69

Merged
mathias merged 1 commits from ci/pr-trigger-68 into main 2026-10-05 07:25:38 +00:00
Owner

Closes #68. Adds a pull_request trigger so Lint/Test/Vet runs on the PR head. build is now limited to push events, and deploy already was, so a PR doesn't build an image, doesn't write to infra and doesn't get access to secrets.

This PR tests itself: it should show a CI run with only the check job. If it does, the change works.

🤖 Generated with Claude Code


Created via git-mcp on behalf of @mathias

Closes #68. Adds a `pull_request` trigger so Lint/Test/Vet runs on the PR head. `build` is now limited to `push` events, and `deploy` already was, so a PR doesn't build an image, doesn't write to infra and doesn't get access to secrets. **This PR tests itself:** it should show a CI run with only the check job. If it does, the change works. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- _Created via git-mcp on behalf of @mathias_
mathias added 1 commit 2026-10-05 05:13:29 +00:00
ci: run the check job on pull requests
CD / Lint / Test / Vet (pull_request) Successful in 15s
CD / Build & Import (pull_request) Skipped
CD / Deploy via GitOps (pull_request) Skipped
0c78df5128
A PR never got a CI run: the workflow triggered only on push to main
and on tags, so the risk/medium+ PR lane merged on local task check
alone (PR #66). pull_request now runs Lint/Test/Vet on the PR head;
build is gated to push and deploy already was, so a PR builds no
image, writes nothing to infra and sees no secrets.

Closes #68

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mathias merged commit c3f8d6c002 into main 2026-10-05 07:25:38 +00:00
Sign in to join this conversation.