Handwerk, Qualität & Teams · Einsteiger

Codeprüfung, die verbessert statt verzögert

In einem Satz: Eine gute Prüfung ist klein, schnell und auf Korrektheit und Wartbarkeit fokussiert — nicht auf Stilvorlieben, die ein automatisches Werkzeug durchsetzen sollte.

Größe

Eine Merge-Anfrage von zweihundert Zeilen bekommt echte Kommentare; eine von zweitausend bekommt „sieht gut aus“. Teilen Sie. Können Sie nicht teilen, schreiben Sie in die Beschreibung einen empfohlenen Lesepfad.

Worüber man kommentiert

Korrektheit, Randfälle, Sicherheit, Leistung an sensiblen Stellen und Klarheit für den, der es in einem Jahr liest. Stil, Abstände und Importreihenfolge — für ein automatisches Werkzeug, nicht für eine Person.

Formulieren Sie Kommentare als Frage oder Vorschlag und markieren, was blockiert und was nur Meinung ist. „Blockierend: das erlaubt Zugriff auf einen anderen Nutzer“ ist klarer als ein höflicher Hinweis.

Antwortzeit

Eine Prüfung, die zwei Tage wartet, stoppt Menschen und erzeugt lange Zweige. Setzen Sie eine Norm — etwa binnen eines halben Tages antworten — und behandeln sie wie jede Teamverpflichtung.

Im Detail

Fügen Sie der Prüfungsliste Punkte hinzu, die schwer automatisch durchzusetzen sind: Wurde ein Test hinzugefügt, ist die Änderung rückwärtskompatibel, wurde die Dokumentation aktualisiert und was passiert bei einem Fehlschlag. Diese vier Fragen fangen den Großteil des Vergessenen.

Prozess & TeamsDokumentation