#16 Code Reviews: Nützlich oder bremsen nur ein gutes Team?

Code Reviews: Jeder will schnelles Feedback, doch niemand hat Zeit dafür - Eine Hassliebe.

Eine Komponente im Alltag jedes Software Engineers. Egal ob Junior, Senior oder Staff-Engineer. Jeder erstellt Code Reviews und kommentiert die Arbeit von den Kollegen. Doch wie sehen gute Code Reviews aus? Was gehört hinein, was bleibt besser draußen? Wie viel Reviewer machen Sinn? Wie geht man mit Nitpicking-Kommentaren und Gatekeepern um? Und allgemein: Zieht dieser zusätzliche Schritt nicht die Performance des Teams runter und ist sowieso Overhead?

All das und noch viel mehr in dieser Episode zum Thema Code Reviews.

Bonus: Was Faultiere und Markus Söder mit Code Reviews zu tun haben und warum Blubberwasser den Charakter verdirbt.

Feedback an stehtisch@engineeringkiosk.dev oder via Twitter an https://twitter.com/EngKiosk

Unsere aktuellen Werbepartner findest du auf https://engineeringkiosk.dev/partners

Links

• Google Fellows: https://www.newyorker.com/magazine/2018/12/10/the-friendship-that-made-google-huge
• Code Owners File: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners
• Software Repository Mining: https://en.wikipedia.org/wiki/Mining_software_repositories
• Round Robin Code Review Assignment: https://docs.github.com/en/organizations/organizing-members-into-teams/managing-code-review-settings-for-your-team
• Inner Source: https://de.wikipedia.org/wiki/Inner_Source
• Suggesting Changes on GitHub: http://haacked.com/archive/2019/06/03/suggested-changes/
• Are Pull Requests Holding Back Your Team?: https://betterprogramming.pub/are-pull-requests-holding-back-your-team-e8aec48986c2
• Extrem Programming: https://de.wikipedia.org/wiki/Extreme_Programming

Sprungmarken

(00:00) Intro

(01:23) Amsterdam, Hipster und Craft-Bier

(02:04) git rebase, history rewrite, force push und Kollaboration

(04:54) Saubere git history und Merge-Commits

(05:17) Code Reviews

(02:05) Was ist denn ein Code Review?

(07:02) Muss der Code Reviewer immer mehr Erfahrung haben?

(08:10) Was wir von Juniors im Job und bei Code Reviews lernen können

(09:37) Was ist denn ein Google Fellow?

(10:37) Code Reviews als Mittel fürs Onboarding

(11:35) Frühes pushen der Arbeit ist wichtig für frühes Feedback

(13:15) Wo sollen die Informationen zu meiner Änderung stehen? Im Pull Request oder in der Git Commit message?

(15:04) Wen lade ich zu meinem Code Review ein? Code Owners file und Software Repository Mining

(17:34) Worauf soll ich als Code Reviewer acht geben und konzentrieren?

(20:09) Pro-Aktivität in Code Reviews mit aktiven Vorschlägen und Fragen stellen

(23:33) Muss man jedes Feedback aus dem Code Review annehmen? Lass das Ego vor der Tür und Gate-Keeper

(26:37) Kommunikationskonflikte in Code Reviews: Video Calls vs. geschriebene Kommunikation

(28:14) Nitpicking, Code-Formatting, Styleguides, Linter und Automatisierung

(32:11) Mindern Pull Requests und Code Reviews die Performance von Teams?

(36:56) Schnelligkeit bei Facebook und Booking: Schnelles Mergen und Code Reviews danach

(39:01) Auslagerung der Verantwortung bei Code Reviews und Blameless Kultur innerhalb der Firma

(42:38) Technische Mittel für sichere Deploys: Feature Flags, Canary Deployments, Dark Launches und Blue-Green-Deployments

(46:08) Wie schnell sollte ein Code Review vom Team gereviewed werden?

(53:00) Code Reviews in der Firma vs. Code Reviews in Open Source und die Motivation

(54:34) Konsequenzen bei nicht eingehaltenen Service Level Agreements (SLA)

(57:28) Outro

Hosts

• Wolfgang Gassler (https://twitter.com/schafele)
• Andy Grunwald (https://twitter.com/andygrunwald)

Engineering Kiosk Podcast: Anfragen an stehtisch@engineeringkiosk.dev oder via Twitter an https://twitter.com/EngKiosk