レビュワーを"憑依"させて Pull Request をセルフレビューする - Konifar's ZATSU

メンバーと1on1をしていると、「うっかりミスが多くて Pull Request で毎回コメントをもらってから気づくのを何とかしたい」という相談を受けることがある。 まず、そういう認識を持てていることが素晴らしい。課題意識があるのであれば、どう補正していくかを一緒に考えることができる。 自分がオススメしているやり方は、レビューを依頼する前に徹底的にセルフレビューすることである。巷でよくやられている方法ではあるが、どういうやり方かを雑に書いておく。 レビューを依頼する前に レビュワーになりきって 自分の Pull Request を自分でレビューしてみる 頭にレビュワーが思い浮かぶのであれば、その人を "憑依" させるイメージ 「この人はここでこういうコメントしそうだな」と思ったら、 先回りして PR上にコメントしておくか、突っ込まれないようにコードやコードコメントを改善する タイトルや説明

[manaten]

これやれる人はやった方がいいんだけど、レビュー必要なのはやれない人なんだよなあ。ただ他人の思考をトレースするのは想像力や視座のトレーニングになると思うので、やれる人はガンガンやるべきにも思う

[odan3240]

コードを書くときから憑依させてる場合もある

[stracciatella]

エンジニアリングじゃなくて一般論だけど、自己批判突き詰めると鬱になる。批判が行き過ぎると自己否定になってしまう

[sainu]

つい最近似たようなアドバイスをしたなぁ

[yamadadadada2]

心の片隅にいつもラバーダックを

[sakurai_youhei]

あとキーボードから一旦離れて無心で10キロくらい歩くという必要性を最近感じている。そんなことしなくてもセルフレビューできるようになりたい。

[mn_kun]

イタコミットは草