プルリクエストを出すとき、レビューするときに気をつけることは何ですか

こう聞かれます

• 「プルリクエストを出すとき、気をつけていることはありますか」
• 「人のコードをレビューするとき、どんな点を見ますか。指摘はどう伝えますか」
• 「レビューで厳しい指摘を受けたら、どうしますか」

面接官が見ている点

• 読む人の手間を減らす出し方 (小さく分ける、説明を書く) を考えているか
• 「何を確かめたか」を自分から書けるか
• 指摘を人への攻撃と受け取らず、コードをよくする話として受け答えできるか

短い答え

• 1 つのプルリクエストには 1 つの目的だけを入れ、読める大きさに分けます。
• 説明には、何をなぜ変えたか、どう確かめたか、確かめていないことを書き、画面の変更なら前と後の画像を付けます。
• レビューでは、直すべき不具合と好みの提案を分けて伝え、受ける側は理由を聞いて直すか、考えを返します。

少し詳しい答え

よくある間違い

• 何日分もの作業を 1 つのプルリクエストにまとめる読む人の負担が大きく、見落としも増えます。目的ごとに分けます
• 説明を「修正しました」だけで出す何をなぜ変え、どう確かめたかが無いと、レビューする人が一から調べることになります
• 好みの違いを、不具合と同じ強さで指摘する相手はどれを直せばよいか分からなくなります。重さを分けて伝えます

深掘りの問い

• 「自分の指摘が間違っていたと分かったら」そのコメントに、誤りだったことと正しい理解を書いて返す、と答えます
• 「レビューが何日も返ってこないときは」小さく分けて読みやすくし、目的と見てほしい所を添えて、声をかける、と答えます
• 「レビュー済みのあとに変更を足したら」足した分を説明に書き、もう一度見てもらってから取り込む、と答えます

関連するコースと用語

• プルリクエストを出す
  コース「Git入門」
• コードレビューの受け方・出し方
  コース「Git入門」
• 良いコミットメッセージの書き方
  コース「Git入門」

用語集で確かめる言葉は、プルリクエスト、コードレビュー、レビュー、CI です。