コードレビュー観点表を作った話

はじめに 今回は、コードレビュー観点表を作った話について少し書かせていただきます。 社内ではGitHubを用いてコードレビューを行っていて、バックエンドの開発においては、コーディングガイドラインも策定しています。 しかし開発において、ガイドラインに書かれている事項が全てではないため、コードレビューを行う際のポイントが自分の中で綺麗に整理しきれていませんでした。 また、ガイドラインの重要なポイントを十分に把握できず、効果的なコードレビューができていない現状がありました。これを改善するために、コードレビューの観点表を作成したことで、コードレビューの質が上がった話についてお話ししようと思います。 問題となっていたこと 一貫性がないレビュー 毎回レビューを行う際に、自分の中のレビューポイントが明確に決まっていなかったため、的確にレビューができていないこと レビューにかかる時間が長い 自分の中でのレ

[lb501]

アメリカ人が何で日本人の優先順位はMUSTしかないんだと言われた事を思い出した。https://dev.classmethod.jp/articles/moscow/

[pmint]

とりあえず否定形やめようぜ。センシティブなデバッグログの作り方は特別に用意するとか、マジックナンバーにはコメントを書くか定数にするとか、理解しがたい命名をしないように理解しやすい命名例を用意するとか。

[vuy]

CopilotとかPR Agent がやってくれそうな内容だった

[gm91]

職場にこれと同じノリでレビューチェックシートがあるけど、内容が総論すぎて全く使い物にならない。

[seal_blog]

これをAIへの指示文にしたらいいレビュー（になりそう

[tg30yen]

コードレビューにおいてマニュアル化できるような内容はAIや静的解析でも大体できるきがする。

[omega314]

まあ、新人にはさっさと当たり前のように理解してほしい項目ではあるので、新人が最初に読む用にはいいかも（経験者の常識は新人の非常識）

[syu-m-5151]

PR-Agent のカスタマイズで良しなにやってくれないかな？

[masatomo-m]

見る範囲全部MUSTで加減の観点（納期、利用シーンから軽減できそうなケース）がなく、殺伐としたコードレビューになりそうだと思ってしまった

[localhost0]

バッドノウハウ

[udofukui]

こういうの重要だよなぁー。そしてブコメも参考になるねぇ

[Flume]

釣られたわ。一部だったら要らんし、抜粋されたのも大した内容じゃ無かった。

[bluescreen]

]

[TriQ]

一部抜粋って全部載せなきゃzennに書く意味ないだろう、、、

[mushus]

似たようなことやってるから見比べたい

[hiroomi]

scaで手を抜けるなら抜いて