コードレビューが機能していないプロジェクトを見ると吐き気がする。

■コードレビューが機能していないプロジェクトを見ると吐き気がする。 複数人で何かを開発する際、新しい機能をガンガン入れていく役割と、それを綺麗に書き直してメンテしやすくする役割がそれぞれ特定の人に偏るというのはよくない兆候であるように思う。なぜならたいていの場合前者の方が評価が高いから。 「○○を実装しました」ってのは分かりやすい。一方、リファクタリングばかりやっていても外から評価されない。当該部分をいじっている人にはありがたがられるかもしれないが、直接関わっていない人にはそのありがたさが全くわからない。 だからこそコードレビューが重要で、チェックインするとき常に最低限の質を保つべき。神経質なぐらいでいい。「動くけど汚い」コードが放置されることを積極的に避けるべき。そして後からリファクタリングが必要とわかったら、なるべくそこを書いた本人が直したほうがいい。そのためにはコードをシェアする全員

[suginoy]

「たいていの場合前者の方が評価が高い」"うちは目立つことしないと評価されないから"って公然と言い放ったマネージャがいたなぁ

[katzchang]

割と後者だけど、レビュア名乗れば評価下がることないよ。それよりも、「チェックイン」という用語が気になる昨今です。