feat(ci): pr-convention-check Drone-Step + Branch-Protection #434

New issue

Closed

opened 2026-06-05 06:54:08 +02:00 by pm-bot · 0 comments

pm-bot commented

2026-06-05 06:54:08 +02:00

Collaborator

Kontext

Folge-Ticket zu #406. Architekten-Empfehlung: A + C kombiniert.

A (Drone-Step pr-convention-check): hard guard
C (PR-Template Closes #-Slot): bereits live in .gitea/PULL_REQUEST_TEMPLATE.md
Anti-Bypass: Gitea Branch-Protection auf main (required status check + Apply to administrators)

Scope

Neuer Step pr-convention-check in .drone.yml, an ci-Pipeline gehängt, event: [pull_request]
Logik:
- Wenn DRONE_SOURCE_BRANCH oder PR-Title eine Issue-Ref andeutet (#\d+ oder N-...-Pattern), dann muss PR-Body oder -Title (Closes|Fixes|Resolves|Behebt|Schließt)\s+#\d+ enthalten
- Sonst pass (Refactor-/Doc-PRs ohne Issue erlaubt)
Manuell: Gitea Repo-Settings → Branch Protection auf main:
- "Status checks must pass before merge" → continuous-integration/drone/pr als required
- "Apply to administrators" toggle

Acceptance

Drone-Step im PR-Build neu sichtbar
PR ohne Close-Keyword (mit Issue-Ref im Branch) failt den Step
PR ohne Issue-Ref (reiner Refactor) passt den Step
Branch-Protection auf main aktiv mit Required-Check + Admin-Enforcement

Aus #406

Vier dokumentierte Convention-Fallen in 6 Tagen (#347, #348, #349, #398) — manuelles Housekeeping bisher.

Fixes #406

## Kontext Folge-Ticket zu #406. Architekten-Empfehlung: **A + C kombiniert**. - A (Drone-Step `pr-convention-check`): hard guard - C (PR-Template `Closes #`-Slot): bereits live in `.gitea/PULL_REQUEST_TEMPLATE.md` - Anti-Bypass: Gitea Branch-Protection auf `main` (required status check + Apply to administrators) ## Scope 1. Neuer Step `pr-convention-check` in `.drone.yml`, an `ci`-Pipeline gehängt, `event: [pull_request]` 2. Logik: - Wenn `DRONE_SOURCE_BRANCH` oder PR-Title eine Issue-Ref andeutet (`#\d+` oder `N-...`-Pattern), dann muss PR-Body oder -Title `(Closes|Fixes|Resolves|Behebt|Schließt)\s+#\d+` enthalten - Sonst pass (Refactor-/Doc-PRs ohne Issue erlaubt) 3. Manuell: Gitea Repo-Settings → Branch Protection auf `main`: - "Status checks must pass before merge" → `continuous-integration/drone/pr` als required - "Apply to administrators" toggle ## Acceptance - [ ] Drone-Step im PR-Build neu sichtbar - [ ] PR ohne Close-Keyword (mit Issue-Ref im Branch) failt den Step - [ ] PR ohne Issue-Ref (reiner Refactor) passt den Step - [ ] Branch-Protection auf main aktiv mit Required-Check + Admin-Enforcement ## Aus #406 Vier dokumentierte Convention-Fallen in 6 Tagen (#347, #348, #349, #398) — manuelles Housekeeping bisher. Fixes #406