より良いコードレビューをするために気をつけていること

Merpay Advent Calendar 2019 の22日目は、メルペイスマート払いチーム/Backend Engineer の @oinume がお送りします。今日はコードレビューについて自分が普段から実践していることを書いてみたいと思います。 はじめに 世の中にはコードレビューをする時の観点については数多く共有されていますが、より良いコードレビューをするためにはどうするのが良いか、というHOWについてのノウハウはあまりシェアされていないような気がしています。そのため、今日は自分なりに心がけているコードレビューのやり方と、ついでに気をつけている観点について書きたいと思います。 Slackを閉じる (これが本当に一番大事だと思っているので最初に持ってきたのですが)私は極端に集中力がないため、SlackのDesktop通知が来るとついついそれが気になって見てしまいます。コードレビューの

[kiyo_hiko]

“APIが冪等になっているか”

[rin51]

> Slackを閉じる

[fullmated]

時間がないときgithub上でなんとなくレビューしてしまうことあった。ちゃんとIDEの力借りよう。

[monoqlock]

panic仕込むの良さそう。けどテストがしっかり書かれてるからこそ出来る事だよねこれは。

[dexia2]

そもそも誰に何をコードレビューしてもらうかというのも大事かなぁと思いました。仕様なのか、設計なのか、実装なのか、動作確認なのか。それぞれ詳しい人は誰で、誰に見てもらうと知識が上手に伝搬されるのか。

[ledsun]

地に足の着いた、いい話だった

[cocodrips]

"panicを入れてテストを全て実行する" なるほどなぁ…

[to4iki]

"Slackを閉じる"

[progrhyme]

手厚い

[braitom]

コードレビューをするときに心がけるとよいことについて。IDEで確認するなどの具体的な方法から、全体像を把握するなどの観点などまで書かれている。

[vvakame]

すごい僕の7倍くらい真摯にレビューしてる…！

[wordi]

影響範囲網羅ならpanicよりruntime.Caller/Callersの方が止まらず全部走るので良いかも、ただ手軽さならpanic

[serihiro]

“ ”

[diveintounlimit]

“その変更で何が変わるのか なぜ変える必要があるのか”これな。全然書かないのが居て毎度ヒヤリングから始めるなどあった。全然読まずに全部聞いてくるのも居た。どっちも不毛だからちゃんと書いてちゃんと読もう

[ingo]

俺たちのoinume

[hirose504]

“APIが冪等になっているか？ メルペイではmicroserviceのAPIを作る時に冪等になっているかどうかを重視しています。”

[techboon]

lintはCIだけだとpushした後になるので、可能ならリポジトリ内にhooks/pre-pushでチェックしてpush止めるのを仕込むコマンド用意しといてメンバみんな適用しております。

[namelaw]

尊いけどレビュアーの負担相当でかそう

[toshi-toma]

そういえばメインの開発はモブプロしかしてないから、ここ2年くらいがっつりしたレビューしてない気がする。レビュー力皆無。

[ryuichi1208]

“APIが冪等になっているか？”これを担保するためのテストコードを書くのが大事ってことか。

[ducktoon]

私はコードレビューという名のマウンティングで心が死にました

[hintoku]

api冪等のテスト観点抜けてたので参考になります

[hiroomi]

”SlackのDesktop通知が来るとついついそれが気になって見てしまいます”雑多をどう片付けるか。scaとも思ったが、環境整備大事っと。

[onichannn]

“コード”

[hkr_I]

よむ

[shiget84]

"APIが冪等になっているか" これ。

[boomerangj]

タイトルで意識高い記事を予想したけど、具体的な内容でよかった

[cl-gaku]

“IDEで見る”