quality-assurance

品質保証

この機能は、変更がマージ前に十分検証されているかを判断するために使用します。これはレビュー担当者の視点です。該当するレビュー指摘の範囲内で不足している証拠を指摘し、ルールを再導出するのではなく、開発者向けのルールにリンクしてください。

全体で使用される重大度ラベル（Critical、Major、Minor）は、プロジェクトのレビュー重大度モデルで定義されています。各レベルの定義、固定の最低基準、判定への対応付けについては、そちらを参照してください。

この文書で使用される「MUST」、「MUST NOT」、「REQUIRED」、「SHALL」、「SHALL NOT」、「SHOULD」、「SHOULD NOT」、「RECOMMENDED」、「MAY」、「OPTIONAL」というキーワードは、RFC 2119 に記載されているとおりに解釈してください。

検証の証拠

verification-evidence.md では、次の内容を説明しています。

• レビュー指摘の範囲内で検証不足を評価する方法（コントリビューターに求められる検証は免除されません）
• tests pass または tested manually という主張を、コマンドや手順、終了ステータス、関連する出力に照らして評価する方法
• 根拠のあるスキップは記録しつつ、未検証として扱う、証拠の妥当性を判断するフロー
• 肯定的な外部レビューの証明がない場合に、スキルの習得と統合後の出力の適合性を分けて評価する方法
• 主張されている義務、成果物、プラットフォーム、観察された段階に対する関連性と最新性を評価する方法
• 成功したチェックが独自にカバーする動作、結果が異なり得たかどうか、チェックから情報が得られなくなるフィクスチャやハーネスの失敗を評価する方法
• 変更された出力面に対応する手動チェック
• 必須チェックのスキップ理由と、それに残るリスクの評価
• Critical または Major の指摘を修正した後の二度目の検証の評価

Lint とフォーマットのゲート

lint-and-format-gate.md では、次の内容を説明しています。

• 作成者がプロジェクトのコード品質ルールに従ってフォーマットと lint のコマンドを実行したこと
• ゲート自体の設定、フック、CI ワークフローに触れる差分、およびゲートが引き続き違反を検出できることを示す証拠
• インラインでの正当化がない新たなインライン lint 抑制がないこと
• 新たに導入された lint エラー、変更されたファイルにおける新しい警告、新しいインライン抑制や回避策としてのキャストの判定

手動検証の証拠

manual-verification.md では、次の内容を説明しています。

• コマンドが成功しても人間が見るようには確認されない、データ駆動型の画面、ルート、実行中のアプリの出力を評価する方法
• 変更がデータ駆動型の画面に関係する場合、作成者がデフォルト以外のコンテンツ状態を確認したこと
• ルーティングの変更で「見つかりません」UI が検証されたこと
• 開発サーバーの出力に新しい警告やエラーがないか確認したこと