コードレビューのコツはアドバイスとクオリティ向上をしないこと
レビューイを尊重する
レビューイの世界観に口出ししない
争いのもとだし、レビューイはその世界観でパフォーマンスを出せるようになってるので矯正すると死んじゃう
(教育的なレビューは除く)
レビューイのコードの「表現の仕方」ではなく「表現しているもの」に注目する
それの品質や性能がヤバければ、それは指摘する
表現の仕方がマズイのなら、あとでまとめてやるのがいいのかなぁ?sta.icon
だったら、メインストリームに突っ込んで開発スピードは落とさずに、マージ後に全体を見通してレビューやリファクタリングをしたり、テスト環境を整えたほうが中長期のスピードと質に貢献できるのでは?と思う
+1sta.icon
いわゆるコーディング規約は、この辺の「個の好み」を殺して「統一的な何か」に従わせることでとりなそうとするところがあるよねsta.icon
sta.icon
議論にも通じるところがあるsta.icon
そもそもコードという創造的なものを複数人でつくってること自体がおかしいんだよなと思うsta.icon*2
まあもうチームで結果出す時代になったからそうしなきゃいけないけど
幸いにもコードは物語とは違うから通用するけど