コードレビューの改善が始まったその先・・・
今日こんなツイートをした。
レビューをしてもらえるのはありがたいけど、指摘の仕方によっては言われた方が困ってしまうこともある。(どう直すべきなのかわからない、何を指摘されているのかわからない。)
↑こんな意見もあった
コードレビューを改善しようと始まったのは最近
以前のチームでのコードレビューは、結構ゆるくて、
1人がコードレビューしたらだいたい終わり、
設計や要件まで把握してコードレビューはしていなかったり、
マージ後にバグが起こったり、
レビュー規則は(ほぼ)なく、
新卒はレビューしていなかった。(下っ端だから?→きちんとレビューされたいし、レビューもしたい。)
そんな中で、きちんとレビューされたいし、レビューしていきたいと言ったら、じゃあ改善していこうかとなった。
コードレビューの改善が始まるまでの流れ
具体的には、
1. コードレビューを自分もやってみたいし、改善したいなあとGKPTでぼやっと言う
2. じゃあ何でやらなきゃいけないのかまとめてきてと言われたので、現状の問題点やメリットデメリットなどをまとめる
3. そのドキュメントを見て改善していくかってなったので、まずはレビュー規則(何人がやる?とか誰がやるとか?期限とか)を固める
4. 次に統一されていなかったコーディング規則(どうコードを書くかの基準)をまとめる
5. 上記2つをチームメンバーで話し合って決める
6. レビューの改善が始まる
レビューが始まったので次のフェーズは
そこから約1ヶ月ぐらいたって、徐々にレビューの改善が始まっていった。
以前提案する際に、コードレビューについて色々調べていたら、よくあった記事が、コードレビューの際に気をつけることとか、コードレビューにおけるレビュアー側のアンチパターンとかだった。
レビューというコミュニケーションにおいてこうしたらいいよねってことだと思うけど、するのもされるのも人間だからそれが難しいし、戸惑うんだろうなとも思った。
自分たちもそろそろそのフェーズなのかなと思ったので、また今週のGKPTでぼやいてみようと思います!
(noteがマークダウンで書けたら嬉しいなあと初めて書いてみて思いました。)
この記事が気に入ったらサポートをしてみませんか?