Seit dem Umstieg auf Git sind Pull Requests ein obligatorischer Bestandteil unseres Workflows. Kein Code gelangt ohne Review in den Develop-Branch. Nach sechs Monaten teilen wir unsere Erfahrungen.
Prozess¶
Feature Branch, Code, Push, Pull Request. Mindestens eine Reviewer-Genehmigung erforderlich. Kommentare, Korrekturen, Genehmigung, Merge.
Checkliste¶
Erfüllt es die User Story? Unit Tests? Keine Code-Duplizierung? Klare Benennung? Sicherheitsprobleme? Fehlerbehandlung? Lesbarkeit?
Kultur¶
Review ist keine Kritik am Autor, sondern eine Verbesserung des Codes. Konstruktiver Ton. Nicht „das ist falsch”, sondern „hast du diesen Ansatz in Betracht gezogen?”
Ergebnisse¶
Durchschnittlich 2,3 Bugs pro PR vor dem Merge gefunden. Wissensaustausch. Konsistente Codebasis. Durchschnittliche Review-Zeit: 30 Minuten.
Regeln¶
- Review innerhalb von 24 Stunden. 2. Kleine PRs (unter 200 Zeilen). 3. Aussagekräftige PR-Beschreibung. 4. Konstruktiver Ton.
Brauchen Sie Hilfe bei der Implementierung?
Unsere Experten helfen Ihnen bei Design, Implementierung und Betrieb. Von der Architektur bis zur Produktion.
Kontaktieren Sie uns