Une pull request est plus facile à examiner lorsque son objectif est clair et que sa différence raconte une histoire cohérente. Le fractionnement des tâches non liées réduit le risque qu'un changement de comportement important soit masqué lors du formatage ou du nettoyage.
Choisissez une limite utile
- Séparez la refactorisation mécanique des changements de comportement lorsque cela rend le résultat plus facile à vérifier.
- Gardez une demande de tirage suffisamment petite pour pouvoir être examinée, mais ne la divisez pas au point que chaque élément n'ait aucun sens en soi.
- Décrivez les dépendances entre les changements et l'ordre dans lequel ils doivent atterrir.
Rendre le chemin de révision explicite
Expliquez le résultat de l'utilisateur ou du système, le choix d'implémentation important et comment vous l'avez testé. Reliez les problèmes liés ou les travaux de suivi plutôt que d’élargir silencieusement la portée actuelle.
Avant de demander une révision, lisez la différence comme si vous n'étiez pas familier avec la tâche. Supprimez le bruit généré et signalez toute pièce intentionnellement incomplète.