IT企業ごとのコードレビュー方式
(insight.infograb.net)-
コードレビューを行う理由:
- 動機付け: レビュアーに迷惑をかけないようにプロ意識を発揮する
- 知識共有: 双方向の知識共有によって組織全体の技術力を高める
- バグ発見: バグを事前に発見できる可能性が高まる
-
Google:
- すべてのコードを
- 社内ポリシーに合ったルールに従って
- できるだけ小さく、頻繁に
- 4時間以内にレビューする(小さなレビューは1時間以内、大きなレビューは5時間以内)
-
Microsoft
- 1日に1回以上レビューする人が39%
-
バンクサラダ
- 小さなPRを頻繁に行う
- レビュアーが簡単に理解し、素早くレビューできるように、変更内容、関連画面、テスト用チェックリストなどをテンプレート化している
-
Woowa Brothers
- MRテンプレートを作って使っている
- MRボットを使って「どのMRが来るのか」という通知が届くようにしている
-
Toss Payments
- 「Good Review」というラベルを付けて、肯定的な表現をすることもある
-
WeMakePrice
- 詳細なレビューと絵文字の使用を推奨している
3件のコメント
マイクロソフト: 1日に1回以上レビューする人が39%
...?
ものすごく少ないんじゃないですか? 意外ですね
会社ごとに違う気がします。うちはすべてのエンジニアが1日に2回以上レビューしていて……笑
これは、見る方によって受け取り方がかなり違うんですよね。^^;