Worauf achtest du, wenn du den React Pull Request von jemandem reviewst?

Zuerst die Korrektheit des Datenflusses: wo der State lebt, ob Effects nötig sind und ehrliche Dependencies haben, und ob Keys stabil sind. Dann die Ränder, die der User sieht, also Loading-, Leer- und Fehlerzustände plus Tastatur- und Screenreader-Zugang. Danach Performance, aber nur wo messbar. Stil und Benennung kommen zuletzt, und was ein Linter oder Formatter entscheiden kann, sollte gar kein Review-Kommentar sein.

Ich lese zuerst die Beschreibung und die Tests, dann den Diff, weil ich wissen will, was die Änderung tun soll, bevor ich beurteile, wie sie es tut. Mein erster Durchgang ist Datenfluss. Wo lebt dieser State, könnte er tiefer leben, ist dieser Effect wirklich nötig oder ist das abgeleiteter State, sind die Dependencies ehrlich oder hat jemand die Lint-Regel leise stummgeschaltet, und sind die Listen-Keys stabil. Der zweite Durchgang sind die Ränder, und dort gehen die meisten Bugs live: Was rendert das beim Laden, wenn die Liste leer ist und wenn der Request scheitert, und kann ich es mit der Tastatur bedienen. Der dritte ist Performance, aber nur wenn ich auf echte Kosten zeigen kann und nicht auf ein Bauchgefühl. Ich markiere Kommentare als blockierend oder als bloßen Gedanken, denn unmarkierte Nitpicks legen Pull Requests tagelang lahm. Und zu Formatierung oder Import-Reihenfolge sage ich nie etwas, wenn das wichtig ist, gehört es in den Linter und nicht in meine Meinung.