『Looks Good To Me 〜みんなのコードレビュー〜』を読む(Part2以降)

Part1の読書メモ

『Looks Good To Me 〜みんなのコードレビュー〜』を読む(Part1) | kmkkiii

https://www.shuwasystem.co.jp/book/b663838.html AI過渡期の今、コードを書くより読む比重が高くなりそうな気配があり、コードレビューの本に興味を持ったので読んでみる。 ## Part1 コー...

kmkkiii.github.io

Part2以降斜め読みしてメモした内容

• 見習いたいこと
  • TWA(チームワーキングアグリーメント)定義
  • PRテンプレート作成
  • Label付け
• レビューコメント
  • Saved replies使ってコメントにラベル付してるので周知しても良さそう
    • https://github.com/settings/replies
    • https://qiita.com/ucan-lab/items/eeef46cee4b928106d1d
• 小さなPRにする
• PRに専門知識が必要でレビューできない
  • 不足している必要なコンテキストや情報はないかPR作成者が注意する
• ペアプログラミングはレビュープロセスの一部を早期に実行するもので置き換わることはない
  • リアルタイム性には優れているが、履歴が残らない
  • 参加者以外への知識移転の制限
  • 同調傾向による偏った視点
• モブプログラミング
  • 開発者以外の人がモブに加わることでコンテキスト改善が期待できる
    • 一方技術的なチェックが見過ごされがちになるかも
• AI
  • レビューの迅速化
  • レビューの一貫性
    • その時の集中力・注意力による
  • AIの限界
    • コンテキストとドメイン知識理解の難しさ
    • 学習データに依存
    • レビュー担当者の専門性の妨げ
  • PR作成者とレビュー担当者が最初のレビューとしてAIを使用することで、そのフィードバックに対する分析力、批判的思考、判断力を磨くことができる
    • セルフレビュー
      • difit
        • https://github.com/yoshiko-pg/difit
  • サービス
    • CodeRabbit
    • Qodo
    • What The Diff
    • GitHub Copilot