Spring GDS 25. Jubiläum
Ein Logistikunternehmen, das in 190 Länder versendet, hat etwas gebaut, um an sich selbst zu liefern.
Code Review
Was ist Code Review?
Code Review ist der Schritt, in dem ein anderer Entwickler eine Änderung liest, bevor sie in die Codebasis gemerged wird. Er prüft, ob sie tut, was sie behauptet, ob sie zum Rest des Systems passt und ob sie in sechs Monaten noch jemand verstehen wird. Dann gibt er sie frei oder bittet um Änderungen.
Die meisten Teams machen das über einen Pull Request. Der Autor schlägt eine Änderung vor, ein Reviewer kommentiert, beide gehen hin und her, und die Änderung wird gemerged, sobald sie standhält. Automatisierte Prüfungen laufen parallel: Linter, Typprüfungen und die Testsuite fangen die mechanischen Probleme ab, damit der Mensch sich auf die Urteilsfragen konzentrieren kann. Ein Reviewer bemerkt, dass eine neue Datenbankabfrage kein Limit hat und eine Tabelle scannen würde, die täglich wächst. Die Tests waren grün. Der Code funktionierte. Unter echtem Verkehr hätte er die Produktion dennoch lahmgelegt, und genau das ist die Art Sache, die Review fängt und Automatisierung verfehlt.
Gut gemacht, verbreitet sie Wissen ebenso, wie sie Bugs fängt. Jetzt verstehen zwei Personen die Änderung statt einer. Schlecht gemacht, wird sie zum Gummistempel oder zum Ort für Erbsenzählerei. Der Unterschied liegt darin, ob das Team sie als gemeinsame Verantwortung für Qualität behandelt.
Code Review bei Dallonses
Nichts erreicht bei Dallonses die Produktion, ohne dass ein anderer Entwickler es gelesen hat. Das ist keine Formalität, die wir erwähnen, um gründlich zu klingen. So bleibt die Arbeit gut, wenn das Team schnell ist und die Menschen wechseln. In der Review werden Standards vermittelt, nicht in einem Dokument, das niemand öffnet.
Wir halten Reviews auf das Wesentliche fokussiert. Korrektheit, Klarheit und ob die Änderung das System leichter oder schwerer lebbar macht. Die mechanischen Prüfungen laufen automatisch in unseren CI/CD Pipelines, sodass Reviewer ihre Aufmerksamkeit den Entscheidungen widmen, die eine Maschine nicht treffen kann. Wenn wir neben den Entwicklern eines Kunden arbeiten, werden unsere Reviews zu einem Weg, zu teilen, wie wir denken, und ihre schärfen unsere. Dieser Austausch ist Teil davon, warum Teams aus einem Projekt stärker hervorgehen, als sie hineingingen.
Möchten Sie ein zweites Paar erfahrener Augen auf Ihrem Code? Sprechen wir.
Verwandte Dienstleistungen
Verwandte Arbeiten
Vizitio
Eine Marke in ein funktionierendes Geschäft verwandeln.
Access Ticket
Eine halbe Million Menschen. Eine App. Null Chaos.
Bereit zum Zusammenarbeiten?
Termin buchen
