良さそうなレビュースタイルはなんだろ - futoaseなんだろな インターネッツで転がってたつぶやきなどから これがいいのかなと思ったのをまとめとこ インデントなどコードフォーマットが基本的なチーム内規約から外れている場合 その人の席にいって指摘する。github上での指摘は1度で抑える。 既存のコードフォーマッタで解決できないか考える 意図の読み取れないクラス名・メソッド名が付けられている場合 チームメンバが相談の上実装しているなら自分も把握できると思うので、「xx classってこういうこと？」とコメントしてそうであれば「んじゃちょっと相談しにそっちいくわ」みたいに軽く行く 無意味なものがある場合、多発するならレビューかペアプロする 自分が未知の仕様が混ざっている場合 即口頭でその人まで会話しに行って納得できるまで話す(5分未満ね) 実コードに対して理想を語るコメントがたくさん付いている場合 「飯食いにいきますか？」の一言で、「俺の理想的
