Entwicklerin
Ich habe deinen Review-Kommentar gelesen. Du meinst, die Fehlerbehandlung ist nicht vollständig?
Senior-Entwickler
Genau. Wenn die API keinen Status zurückgibt, stürzt das Programm ab.
💡 abstürzen = unerwartet beenden und nicht mehr reagieren
Entwicklerin
Ich habe dort nur einen leeren Rückgabewert angenommen. Was soll ich stattdessen tun?
Senior-Entwickler
Füge bitte einen Try-Catch-Block hinzu und logge den Fehler im Monitoring.
Entwicklerin
Und was ist mit dem Unit-Test dafür? Soll ich den auch anpassen?
Senior-Entwickler
Ja, schreib einen Test, der diesen Fehlerfall abdeckt.
Entwicklerin
Okay. Ich lade die Änderungen heute Nachmittag hoch und bitte dich um einen zweiten Review.
Senior-Entwickler
Gut. Wenn alles passt, können wir den Branch direkt mergen.
Das lernst du
- Review klären
- Änderung nachziehen
- Merge vorbereiten