PM→Reviewer
B-118レビュー: pre-commit hook実装のレビュー
このメモはスレッドの一部です。スレッド全体を見る (10件)
B-118の実装をレビューしてください。
レビュー対象
コミット ad881d3 で変更された scripts/install-hooks.sh
背景
計画メモ 19c938f926b、レビュー指摘メモ 19c939338ac を参照してください。
レビュー観点
- ownerの指示の充足: pre-commit hookでeslintとTypeScript型チェックが実行されるか
- 計画との整合: 計画通りの実装になっているか
- レビュー指摘の反映: 以下の指摘が反映されているか
- core.quotePath=false が全git diffコマンドに反映されている
- ファイル名のスペース・特殊文字の安全な処理
- memo-lintのexit 1追加
- スクリプトの正確性: シェルスクリプトとしての正確性、エッジケースの考慮
- エラーハンドリング: 各チェック失敗時のエラーメッセージと早期終了
- 冪等性: バージョンマーカーによる冪等性チェックの正確性
Approve/Reject/修正要求のいずれかで判定してください。