Code Reviews über Pull Requests sind Standard in den meisten Teams. Aber was genau wird geprüft und wie läuft das ab?

Schritt 1: Pull Request analysieren

Die Beschreibung lesen. Was soll der Code erreichen? Welches Problem wird gelöst? Bei einem neuen API-Endpoint etwa: Welche Daten werden zurückgegeben?

Schritt 2: Änderungen durchgehen

Die Diff-Ansicht zeigt alle Änderungen. Grün ist neuer Code, rot ist gelöschter Code. Bei größeren PRs die Dateien nach Wichtigkeit sortieren – erst die Logik, dann die Tests.

Schritt 3: Logik überprüfen

Macht der Code was er soll? Gibt es Edge Cases die nicht behandelt werden? Bei einer Validierungsfunktion: Was passiert mit leeren Strings oder Null-Werten?

Schritt 4: Codequalität bewerten

Ist der Code lesbar? Sind die Variablennamen sprechend? Gibt es doppelten Code? Eine Funktion mit 200 Zeilen sollte aufgeteilt werden.

Schritt 5: Tests prüfen

Gibt es ausreichend Tests? Decken sie die wichtigen Fälle ab? Bei einem Zahlungssystem müssen Success- und Failure-Szenarien getestet sein.

Schritt 6: Kommentare hinterlassen

Konkrete Verbesserungsvorschläge formulieren. „Diese Schleife könnte durch ein Array-Filter ersetzt werden" ist besser als „Code optimieren".

Schritt 7: Approve oder Request Changes

Bei kleineren Anmerkungen approven und dem Entwickler vertrauen. Bei größeren Problemen Changes anfordern und auf die Überarbeitung warten.