CI never runs on pull requests — the risk/medium+ PR lane merges with no CI evidence #68

Closed
opened 2026-10-05 05:13:02 +00:00 by mathias · 0 comments
Owner

Found on 2026-10-04 while merging PR #66 (#65).

This repo's workflows trigger only on push to main and on tags. A pull request therefore never gets a CI run. The gatekeeper merged #66 with a local task check (exit 0) as its gate, and the harness flagged that as a CI bypass. CI after the merge (run 9316) was green, but that green came after the merge, not before it.

So the PR lane that rule 5 requires for risk/medium+ work has no CI on the change it is reviewing. That matters most here, because gitea-mcp is the write path the rest of the estate uses to reach Gitea.

Fix: add pull_request to the CI workflow's triggers, for the jobs that need no secrets (lint, vet and test). Keep the build and deploy jobs on push and tags only.

Done when: a PR shows a CI run on its head sha before it merges.

Other repos may have the same gap. Worth a grep across .gitea/workflows for workflows that have no pull_request trigger.


Created via git-mcp on behalf of @mathias

Found on 2026-10-04 while merging PR #66 (#65). This repo's workflows trigger only on `push` to main and on tags. A pull request therefore never gets a CI run. The gatekeeper merged #66 with a local `task check` (exit 0) as its gate, and the harness flagged that as a CI bypass. CI after the merge (run 9316) was green, but that green came after the merge, not before it. So the PR lane that rule 5 requires for risk/medium+ work has no CI on the change it is reviewing. That matters most here, because gitea-mcp is the write path the rest of the estate uses to reach Gitea. **Fix:** add `pull_request` to the CI workflow's triggers, for the jobs that need no secrets (lint, vet and test). Keep the build and deploy jobs on push and tags only. **Done when:** a PR shows a CI run on its head sha before it merges. Other repos may have the same gap. Worth a grep across `.gitea/workflows` for workflows that have no `pull_request` trigger. --- _Created via git-mcp on behalf of @mathias_
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mathias/gitea-mcp#68