レビュー
種類
教育レビュー
全体に共有
底上げ
SECIモデルでいう共同化・表出化
マネジメントレビュー
マネジメント判断に必要な情報を集める
ピアレビュー
成果物の欠陥と改善のポイントを探す
Pull Request Reviewは主にこれ
振り返り
将来に活かす
ステータスレビュー
関係者で進捗を確認
目的
検査
学習
強化
関連
2024 https://github.blog/developer-skills/github/how-to-review-code-effectively-a-github-staff-engineers-philosophy/
GitHubのStaff Engineerによる効果的なレビューについて
自分のコードを完成させるためにさらに不明な時間を費やすよりもshipにより近いコードをゴールラインまで到達させたい
フィードバックを早く提供すればするほど、そのフィードバックに早く対処してバグを潰したり機能を出荷したりできる
レビューはキャリアを形成するのに重要
チームメイトに影響のある活動
通常、昇進するには高い影響力を示す必要がある
セルフレビューする
レベル差/ドメイン知識差によってわからないことを聞く
マージ後のレビューも推奨する
I’m also big on junior developers asking senior developers questions in code review, even if they think their question is silly or has an obvious answer.
2022-09
コードレビューはつまらないから丁寧なプルリクエストでチームの生産性向上を目指す
2022年までの記事や書籍の考え方がまとまっている
2020 コードレビューの目的と考え方
コードレビューの目的を大小2つに整理し、優先度順のチェックリストが提示されている
Pull Request Review
https://speakerdeck.com/takaking22/teams-review-for-quality