Das Theater der Pull Requests und Code Reviews
Saša Jurićs Vortrag auf der Goatmire Elixir Conf verwandelte die Code-Review in eine fesselnde Erzählung. Er hob das häufige Problem zu großer und komplexer Pull Requests (PRs) hervor, die zu oberflächlichen Reviews, Sicherheitsrisiken und schwer wartbaren Codebasen führen. Die wichtigste Erkenntnis: Überprüfbare PRs sollten idealerweise 5-10 Minuten zur Überprüfung benötigen, idealerweise unter 300 Zeilen Code. Dies wird erreicht, indem prägnante, erzählerische Commit-Nachrichten erstellt werden, die die Argumentation und die Schritte jeder Änderung klar erklären. Das Aufteilen großer Features in kleinere PRs und die Verwendung von Tools wie `git fixup` zur Aufrechterhaltung eines sauberen Commit-Verlaufs sind entscheidend für eine effiziente Code-Review und einen qualitativ hochwertigeren Code. Der Vortrag betonte, dass "Ich verstehe es nicht" besser ist als ein bedeutungsloses "LGTM."
Mehr lesen