Code-Review-Methoden in IT-Unternehmen

• Gründe für Code Reviews:

  • Motivation: professionelles Verantwortungsbewusstsein zeigen, um dem Reviewer keine zusätzliche Last aufzubürden
  • Wissensaustausch: bidirektionaler Wissensaustausch steigert die technische Kompetenz der gesamten Organisation
  • Fehlererkennung: höhere Wahrscheinlichkeit, Fehler frühzeitig zu entdecken

• Google:

  • Jeder Code wird
  • nach Regeln geprüft, die den internen Richtlinien entsprechen
  • möglichst klein und häufig
  • innerhalb von 4 Stunden reviewt (kleine Reviews innerhalb von 1 Stunde, große Reviews innerhalb von 5 Stunden)

• Microsoft

  • 39 % reviewen mindestens einmal pro Tag

• Banksalad

  • arbeitet häufig mit kleinen PRs
  • erstellt Templates für Änderungen, relevante Screens und Checklisten zum Testen, damit Reviewer den Inhalt leicht verstehen und schnell reviewen können

• Woowa Brothers

  • verwendet erstellte MR-Templates
  • nutzt einen MR-Bot, damit Benachrichtigungen dazu eintreffen, „welche MRs hereinkommen“

• Toss Payments

  • vergibt auch das Label „Good Review“, um positive Formulierungen zu fördern

• Wemakeprice

  • empfiehlt ausführliche Reviews und den Einsatz von Emoticons

[kuber]

Microsoft: 39 % überprüfen mehr als einmal pro Tag Reviews
...?
Ist das nicht extrem wenig? Unerwartet.

[superwoou]

Das scheint je nach Unternehmen unterschiedlich zu sein. Bei uns machen alle Engineers mindestens zweimal am Tag Reviews ... haha

[ironlung]

Das wird offenbar von jedem, der es liest, anders aufgefasst. ^^;