設計
- 横展開は必要か
コーディング時
- 簡単なプログラム設計を実施したか
DBアクセス
- N+1は発生しないか
- DBアクセスのパフォーマンスに問題ないか
- SQLインジェクションは発生しないか
特殊な条件での動作確認
- レスポンシブ対応の動作確認をしたか
- ダークモード時の動作確認をしたか
- 一通りのパターンで画面を動かしたか
リファクタリング
- エラーハンドリング
- 将来的な変更に強い構造か
- 自分で読んで理解できるか
- 他の人が読んで理解できるか
- コードの構造の違和感を見逃していないか
- 動作の違和感を見逃していないか
- リファクタリングに時間をかけたか
- 時間をおいて自分のソースを見直したか
プルリクエスト時
- テストを書いているか
- コーディング規約
- コメント(Codeiumも活用する)
- 横展開は実施したか
- 不要なコードが残っていないか
- IDEの警告が出ていないか
- テスト実行結果を張り付ける
- Pipeline実行結果を張り付ける
- 修正対象ソースのカバレッジを張り付ける
- 修正内容の要約を記載する
- 時間をおいて修正内容の要約が分かりやすいか見直したか
- PRに関連するWork Itemを追加する
- TaskをResolvedにしたか
マージ後
- チケットに作業時間を追記する
- チケットをクローズする(必要な場合は親チケットもクローズ)
- 設計書は最新化したか