Open2023/10/11にコメント追加1ペアレビューから学んだことコードリーディングコードレビュー腰痛エンジニア2023/10/11まずはPRの概要から見た 概要にissueが紐づけてあったので見た issueからPRがどのような問題を解決したいかを把握 issue解決のコアとなる部分を想像する コアとなる箇所を探す 上流の関数から見ていく 下流の関数には飛ばずにDocコメントや関数名から動作を想像するにとどめる (外部ライブラリの場合はドキュメントがありそうなのですぐ調べた方が良さそう) 自分は何を見ていたかすぐ忘れるのが困ったな〜
腰痛エンジニア2023/10/11まずはPRの概要から見た 概要にissueが紐づけてあったので見た issueからPRがどのような問題を解決したいかを把握 issue解決のコアとなる部分を想像する コアとなる箇所を探す 上流の関数から見ていく 下流の関数には飛ばずにDocコメントや関数名から動作を想像するにとどめる (外部ライブラリの場合はドキュメントがありそうなのですぐ調べた方が良さそう) 自分は何を見ていたかすぐ忘れるのが困ったな〜