Bisher haben Sie Ihre eigene Kladde gebaut und geprüft. In dieser Lektion prüfen Sie die Kladde einer Person aus Ihrem Kreis (Kollegin, Lerngruppe, öffentliche Lerngemeinschaft), und diese Person prüft Ihre. Finden Sie niemanden, prüfen Sie Ihre eigene Kladde nach einer Pause mit denselben Fragen, als wäre sie fremd. Ein Architektur-Review ist keine Note und kein Code-Review in groß. Es fragt, ob ein System seine eigenen Ziele erreicht und ob seine Entscheidungen tragen: Hält die Isolation, was das Qualitätsszenario verspricht? Lässt sich ein Büro wirklich zurückholen? Würde eine neue Kollegin aus der Dokumentation verstehen, wie ein Angebot durch das System läuft?

Das Review ist Teil des Abschlussprojekts. Für das Architektur-Review mit Checkliste (4.9.7) liegen das Review, das Sie geschrieben haben, und Ihre Antworten auf das Review Ihrer Arbeit bereit. Beides zählt für Ihren eigenen Lernerfolg: wie genau Sie eine fremde Architektur lesen, und wie Sie mit Kritik an Ihrer eigenen umgehen.

So läuft das Review

Je zwei Teilnehmende prüfen sich gegenseitig, oder drei im Kreis (A prüft B, B prüft C, C prüft A); im Kreis entsteht kein Tauschhandel nach dem Muster „du schonst mich, ich schone dich“. Der Ablauf hat vier Teile:

Teil Wer Dauer Ergebnis
1. Der 30-Minuten-Test Sie lesen die fremde docs/, ohne Code 30 Minuten lesen, 5 Minuten erklären erste Befunde zur Dokumentation
2. Prüfen Sie lesen Code, lassen Tests laufen, prüfen Nachweise etwa 75 Minuten Notizen mit Belegen
3. Schreiben und besprechen Sie schreiben das Review (etwa 30 Minuten), dann 30 Minuten Gespräch etwa 60 Minuten docs/reviews/review-1.md im fremden Repository
4. Antworten die geprüfte Person bis zur Checkliste in 4.9.7 jede Zeile beantwortet

Drei Regeln gelten für alle Teile:

  • Die Arbeit wird geprüft, nicht die Person. „Der Worker hat mehr Rechte als die Rechtetabelle“, nicht „du hast die Rechte vergessen“.
  • Fragen vor Urteilen. Was nach einem Fehler aussieht, ist oft eine Entscheidung, die Sie nicht kennen. Suchen Sie das ADR; gibt es keins, ist genau das der Befund.
  • Kein Befund ohne Beleg. Eine Datei mit Zeile, ein Test, eine Messung, ein Link. „Die Isolation wirkt unvollständig“ ist ein Gefühl; „GET /api/tenders/{id}/documents steht als Ausnahme ohne Grund in IsolationTests“ ist ein Befund.