mex

2021.03.03

レビュー依頼が多くてだんだんとコミュニケーションが雑になってるというか、コードに対してシビアになりすぎている ここを直すならこっちも直してください。せっかくなのでみたいな。これはあまり良いふるまいではないので気をつける。実装者に余計なプレッシャーを与えるし、実装レベルが急に上がるわけでもないし。レビュアーが気持ち良くなるだけで利がない。

過去に参画していたプロジェクトでレビューの文面や指摘がかなり厳しい人がいて、PRをだすのがとても嫌になった記憶がある。あまりに態度がきついのでレビューとはこういうものなのかと思って調べ、見つけたのが↓の記事。

loading

こっち日本語訳

loading

結局プログラマーは人間でコードレビューはコミュニケーションの延長で、実装者とレビュアーは状況に応じて逆転する。厳しい言葉で殴り合っていたら身が持たない。マウント取るためにレビューしているわけではない。自戒。