プルリクとかレビューについて尊敬できるエンジニアに相談してみた話 - まえとうしろ誰かと開発することを忘れかけていて、 もぅﾏﾁﾞ無理。。。とりあえずLGTMしょ。。。 となりかけてしまったので、尊敬できるエンジニアである夫に相談してみました。 世の中の正解かは分からないですが、私は参考にしたいと思ったのでまとめます。 レビューの仕方がわからないです！！どこから見たらいいかわからないです！！ プルリクでは「意図」がわかるようにしたほうがいい。 夫が所属しているチームでは下記の内容をプルリクの説明(description)に書いているそうです。 なぜこの変更を入れるのか バグだったらどういう問題があったか、機能だったらこれを入れたら何が改善されるのか(なにが嬉しいの？) 変更概要 変更概要を読むとdiffを見たときに意味がわかるように こういう理由でこういう実装になっている、こういう理由でここは直さなくてもいい、などを書く 関連Issue チケット(issue)へのリンク
