#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