review

レビュー

変更が完了したとみなされるのは、構成、差分、ドキュメントの3つの観点すべてに合格してからです。それぞれに専用のスキルがあります。このスキルが担うのは、それらではカバーされない部分、つまり実行順序、各スキル間の反復、そして最終判定だけです。

ワークフロー

1. 範囲の確認。 レビュー対象の差分を確定し、内容のある変更であることを確認します。些細な編集なら、レビュー全体を省略します。
2. 構成 — refactor-clean。 最初に実行します。構成の変更が容易なうちに、また移動する予定の差分をレビューする前に行います。
3. 差分 — code-review。 構成が確定してから監査します。構造の変更が必要な指摘があれば、次へ進まずステップ2に戻ります。
4. ドキュメント — write-docs。 この変更が触れたドキュメントを更新します。その後、ルートのREADMEから各ドキュメントまでリンクをたどり、パス上のすべてのリンクが引き続き有効であること、またハブから末端までの流れが順序よく読めることを確認します。
5. 報告。 3つのパスをまとめて1つの判定を示します。各パスで見つかったこと、変更したこと、変更せずに残したこととその理由を報告します。すべてのパスに問題がないか、解決済みになった場合にのみ完了です。

ルール

• 各パスを開始する際は、そのスキルを読みます。ルールはここではなく、各スキルに記載されています。
• 順序が決めるのは提示の順番であり、開示のタイミングではありません。後のパスが担当する指摘であっても、見つけた時点ですぐに共有します。
• 連鎖させず、反復します。後のパスでの修正によって前のパスを再度確認する必要が生じたら、そこに戻ります。パス全体を実行して新たな指摘がなくなるまで続けます。