28 ポイント 投稿者 ironlung 2023-09-25 | 3件のコメント | WhatsAppで共有
  • コードレビューを行う理由:

    • 動機付け: レビュアーに迷惑をかけないようにプロ意識を発揮する
    • 知識共有: 双方向の知識共有によって組織全体の技術力を高める
    • バグ発見: バグを事前に発見できる可能性が高まる
  • Google:

    • すべてのコードを
    • 社内ポリシーに合ったルールに従って
    • できるだけ小さく、頻繁に
    • 4時間以内にレビューする(小さなレビューは1時間以内、大きなレビューは5時間以内)
  • Microsoft

    • 1日に1回以上レビューする人が39%
  • バンクサラダ

    • 小さなPRを頻繁に行う
    • レビュアーが簡単に理解し、素早くレビューできるように、変更内容、関連画面、テスト用チェックリストなどをテンプレート化している
  • Woowa Brothers

    • MRテンプレートを作って使っている
    • MRボットを使って「どのMRが来るのか」という通知が届くようにしている
  • Toss Payments

    • 「Good Review」というラベルを付けて、肯定的な表現をすることもある
  • WeMakePrice

    • 詳細なレビューと絵文字の使用を推奨している

3件のコメント

 
kuber 2023-09-25

マイクロソフト: 1日に1回以上レビューする人が39%
...?
ものすごく少ないんじゃないですか? 意外ですね

 
superwoou 2023-10-01

会社ごとに違う気がします。うちはすべてのエンジニアが1日に2回以上レビューしていて……笑

 
ironlung 2023-09-27

これは、見る方によって受け取り方がかなり違うんですよね。^^;