アプリケーションの保守をやっていると、色んなPull Requestを見る機会がある。レビューする/しないは関係ない。自分の変更を入れる前に過去の実装経緯を確認し、デグレリスクがないか・暗黙的なコンテキストがないか確認したいのだ。もちろんレビュアーをやることもある。解読に時間がかかるPull Requestの傾向が見えてきたのでメモする。 変更点が多い 散々言われているので割愛。コードフォーマッタを途中で導入する場合、コードフォーマット用のPull Requestを作成してほしい。 説明書きやコメントが何もない 保守する側からすると最悪。Issueも真っ白だとヘイトが増してしまう。「今だけ乗り切れば良い」とか、「自分が分かってれば良い」という考えなのかもしれない。ただ、いつかは担当は変わる。そしてコードはメンテされる時間のほうが長い。 せめて何故この変更が必要なのかに関する説明書きは欲しい