実務シチュエーション4 択難易度 標準

レビュアーとして適切な対応は？

 payment_api:   base_url: https://pay.example.com-  timeout_ms: 3000+  timeout_ms: 300

正答

1. 1 行だけの変更で急ぎなので、差分の確認は省いて Approve する
2. ついでに base_url も環境変数に移すよう、あわせて修正を求める
3. YAML のインデントを 4 スペースにそろえるよう、修正を求める
4. 単位が ms なので 300 は 0.3 秒。30 秒なら 30000 と指摘する

解説

設定の単位はミリ秒なので、300 は 0.3 秒であり、30 秒に延ばすどころか 3 秒から 0.3 秒へ短くなる。反映すると外部 API の呼び出しが次々にタイムアウトし、障害になりうる。PR の説明と差分の内容が一致しているかを確かめるのは、急ぎでも省けない確認である。base_url の移動は PR の目的外の要求で、インデントの好みは急ぎの依頼で優先する論点ではない。

reviewconfig