プルリクでやられると困っちゃうことまとめ - Qiita最近めっきり便利屋さんみたいなポジションになってしまい、サーバーにしろアプリにしろ「プルリクお願いします」と言われることが多くなってきました。 僕もスーパーなエンジニアでは全くないのであれこれ試行錯誤しながらコードを見ているのですが、今回はそんな中でやられると困っちゃう（オブラート10枚くらいで包んだ表現）ことをまとめます。 ちょっと困っちゃうやつ プルリクの説明がコミットログ（しかもテキトーに書いたやつ）の羅列 プルリクのタイトルがない 改行コードが変わったのか、ファイル全体が差分になる 明らかに不要なコメントアウトが残ってる 個人的なデバッグ用のコードが残ってる（sysoutとか） 指摘された部分を修正した際、同じ内容の他の箇所が直っていない 困っちゃうやつ コードに明らかに実装漏れ（空のメソッドとか）があったので指摘したら、「あ、それ別のブランチで対応予定です」って言われる プルリク
