Révision de PR
Passez en revue la surface de modification de la branche, pas l'ensemble du dépôt.
- Établissez le diff :
git diff <base>...HEAD(etgit diff <base>lorsque l'arbre de travail contient des modifications non commitées qui appartiennent à la branche). La branche de base est nommée par l'appelant ; par défaut, utilisezmain. - Vérifiez chaque fragment modifié pour l'exactitude : résultats incorrects sur des entrées limites, chemins d'erreur non traités, et comportement contredisant le nom ou les appelants de la fonction.
- Vérifiez la couverture de tests pour le comportement modifié : toute nouvelle logique sans test exerçant ses cas limites constitue une anomalie.
- Vérifiez la documentation rendue obsolète par le diff : des affirmations dans le README ou la documentation qui étaient vraies sur la branche de base et deviennent fausses après la modification.
- Rapportez chaque défaut trouvé, chacun avec sa sévérité ; incluez les constatations mineures et informatives. Ne rapportez pas de défauts dans des fichiers que la branche n'a pas touchés — ils sont hors de la portée de la pull request.