コードレビューを会話しながら行う取り組み - Hatena Developer Blog

こんにちは。アプリケーションエンジニアの id:itchynyです。 今回は、コードレビューを会話しながら行う取り組みについて紹介します。 コードレビューは大事なコミュニケーションの場です。 コードレビューの効用としては、単純なミスがあるコードをリリースしない・プロダクトのコードの品質をよりよくしていく、あるいはその方策を模索するといったことが挙げられます。 こういったことは当然のことですが、なによりもまず、レビューというのは一緒にプロダクトを作っている仲間とのコミュニケーションの場だと思います。 多くの人は、プロダクトのコードをよくしていきたい、読みやすいコードを書きたい、分かりやすいコードで目的の機能を作りたいといった共通の思いを持っていることでしょう。 コードを書いた人の思いを汲み取りながら、共感したり、譲歩したりしながら、よりよい方法を提示していきます。 それでも時には、どういうコ

[NOV1975]

名前をつけるまでもなく昔から普通に必要に応じて対面レビューってするものじゃない？／なんかこう新しそうに書いているけど、古い知見を古いと言って捨てた結果、車輪の再発明しているのでは？

[nkazuki]

ふむふむ。確かに非同期のレビューで何往復もするより、話したほうが良いことは多い。一方で同期的なコミュニケーションが強い（声の大きい）人が一人勝ちしてしまわないような工夫がどうにかほしいような気もする

[takogawa]

大きな差分がある時にペアレビューをやってみると、実装の意図がいつもよりもよく分かってやってよかったと

[ttanimichi]

これ俺も意識してやってた。隣に座って会話しながらだと無駄な軋轢を生まないから良い

[rohki]

さっき新人とやってた。もの見ながらやりとりする方が早いよね。

[gfx]

なるほど。読んでいて引っかかったところについては書いた人の温度感を確認しながら進めたいことはあるなあ。

[progrhyme]

「ペアレビュー」か。

[braitom]

PRベースでのオンラインレビューばかりしちゃうけどたまにはペアレビューした方が良さそう。

[bou56tcdu]

こーゆーの重要だよね。すいません・・・。

[upamune]

大変参考になる

[tick2tack]

ペアレビュー。ペアコーディングよりかは受け入れられやすそう。

[gologo13]

ペアレビューいいと思うけど、そもそも意図を聞かないと伝わらないコードは残すべきではない

[surume000]

コーデレビューは差分だけの部分最適になりがちなので嫌い。導入するにはそれなりの仕組みが必要

[cl-gaku]

対面レビューっこうじゃないの？と思ったけどやっぱそうだよな

[murasaki11]

いい職場だなぁ

[takuya-a]

はてなではテキスト文化が強いのでGHE上でのやりとりで完結させがちだけど、オフラインでの会話もうまく取り入れていていい試みだなーと

[shngmsw]

まずコードレビューして指摘をメモった上で対面して納得できれば指摘取り下げするとか普通にするもんでは？Redmineにコードレビュープラグイン入れれば電子に残せるし。

[koogawa]

ペアレビューした方が速いこと多いよね

[Palantir]

ぼくもよくコードと話してるよ(？)

[luccafort]

PRレビュー依頼して直接「この部分の実装なんでこうなってるの？」みたいなのをやったりやらなかったりなので決まったタイミングでやるのは確かに良いかも。

[anakahala]

“なによりもまず、レビューというのは一緒にプロダクトを作っている仲間とのコミュニケーションの場だと思います。 ”

[skit_n]

若手相手だと、まず対面でやりとりしたあとに、決まったことを改めてコメントすることはよくやる

[daichirata]

よく分からない時に直接聞きに行く奴 / ライフスタイルの変化からリモートを推進したい身としては、もっと手前の段階からレビューしてそもそものズレを少なくしていきたい

[ryota-murakami]

社内開発なのにご丁寧にPRの体裁整えるのに時間使って生産性落とす文化マジで廃止しろ

[hiro_curry]

これで盛り上がれば離職率下がりそう。

[zentarou]

でもログに残らないんだよなぁ。とはいえレビュー指摘事項Excelは作りたくない。

[shiba_yu36]

最近社内で続々行われている

[side_tana]

はてなのエンジニアとペアレビューしたい！

[non_117]

よさそう

[Pasta-K]

丁寧で良さそう