Ein Code Review ist die Durchsicht einer Änderung durch jemand anderen, bevor sie in den gemeinsamen Stand übernommen wird.
- Code Review
Die Prüfung einer Codeänderung durch eine zweite Person, bevor sie in den gemeinsamen Stand aufgenommen wird.
Beispiel: „Jede Änderung geht durch ein Code Review, auch die von erfahrenen Entwicklern.“Auch: Quelltextprüfung, Vier-Augen-Prinzip.
Wozu es dient
Fehler zu finden ist nur der offensichtliche Teil. Mindestens so wichtig ist, dass Wissen sich verteilt: Nach einem Review kennen zwei Menschen die Stelle, und niemand ist mehr allein für einen Bereich unentbehrlich.
Dazu kommt die Wirkung auf den Code selbst. Wer weiß, dass jemand mitliest, schreibt anders, und zwar meist sorgfältiger. Das liegt am Verfahren und hat mit Misstrauen nichts zu tun.
Was ein gutes Review ausmacht
Kleine Änderungen. Eine Durchsicht von zweihundert Zeilen findet Fehler, eine von zweitausend findet Tippfehler. Wer große Umbauten prüfen lässt, bekommt oberflächliche Rückmeldungen, und das liegt nicht an der Sorgfalt der Prüfenden.
Dazu gehört die Unterscheidung zwischen muss, sollte und Geschmack. Was den Code kaputt macht, blockiert; was anders schöner wäre, ist ein Hinweis. Wer beides gleich gewichtet, macht das Verfahren zur Zumutung.
Was Werkzeuge vorher erledigen
Formatierung, Stilfragen und einfache Fehler gehören zum Linter und zum Formatierer, nicht ins Review. Diese laufen automatisch, und erst danach sieht ein Mensch darauf.
In diesem Projekt übernehmen das Pint und PHPStan, dazu die Testsuite. Was eine Maschine entscheiden kann, soll keine Diskussion kosten.