Une fois que vous avez ouvert une pull request, les relecteurs laissent des commentaires qui permettent d’améliorer votre code avant sa fusion. La résolution de ces commentaires implique de comprendre chaque commentaire, d’apporter les modifications qu’il appelle et de vous assurer que vous avez tout traité.
Comprendre les commentaires de révision
Les réviseurs peuvent laisser des commentaires généraux, commenter des lignes spécifiques et proposer des modifications exactes en tant que suggestions. Les révisions apparaissent dans la chronologie des demandes de tirage pour vous permettre de suivre la discussion et de voir quels commentaires ont toujours besoin d’une réponse. Selon les paramètres du dépôt, un examinateur peut également approuver votre pull request ou demander des modifications que vous devrez apporter avant la fusion.
Avant de modifier quoi que ce soit, lisez chaque commentaire pour comprendre son intention. Un commentaire peut signaler un bogue, poser une question, demander une approche différente ou suggérer une modification que vous pouvez accepter directement.
Si vous avez accès à Copilot, il peut vous aider à interpréter les commentaires de revue et à proposer des correctifs, ce qui est utile lorsqu’une pull request comporte de nombreux commentaires à traiter.
Implémentation des modifications et correction du code
Lorsque vous répondez aux commentaires, vous pouvez :
- Appliquez directement la modification suggérée d’un réviseur, qui valide la suggestion dans votre branche.
- Apportez des modifications plus larges localement et envoyez de nouvelles validations à la branche.
La plupart des retours peuvent être pris en compte en mettant à jour votre code et en poussant de nouveaux commits vers la branche de la pull request. Lorsque vous répondez aux commentaires, marquez les conversations comme résolues afin que vous et vos réviseurs puissiez suivre ce qui est fait et ce qui a toujours besoin d’attention.
Lorsque les relecteurs requis ont approuvé et qu’aucune demande de modification n’est en attente, votre pull request est prête à être fusionnée. Étant donné que la pull request est liée à la branche, chaque nouveau commit met à jour la pull request et relance toutes les vérifications automatisées.
Traiter efficacement les retours
Sur les demandes de tirage plus volumineuses ou plus examinées, quelques pratiques vous aident à répondre rapidement aux commentaires :
- Lot acceptant des suggestions en ajoutant des modifications suggérées à un lot afin que plusieurs modifications acceptées atterrissent dans une validation unique au lieu d’une validation par suggestion.
- Résolvez à l’avance tout conflit de fusion afin de pouvoir effectuer la fusion rapidement une fois la révision de votre pull request terminée.
- Redemandez une révision après avoir apporté des modifications substantielles, afin que les réviseurs sachent que la pull request est prête pour un nouvel examen.
- Comprendre les révisions requises. Lorsqu’une branche nécessite des approbations ou la validation du propriétaire du code, une revue « Demander des modifications » ou une approbation révoquée peut bloquer la fusion tant que le problème n’est pas résolu.
- Suivez les retours hors périmètre en ouvrant une issue renvoyant au commentaire, plutôt qu’en élargissant la pull request.
Outils pour vous aider à résoudre les révisions
Vous n’avez pas à traiter les retours manuellement, seul :
-
Consultez la pull request localement ou ouvrez-la dans GitHub Codespaces pour reproduire les problèmes et tester les correctifs avant de pousser vos modifications.
-
Corrigez les problèmes de sécurité signalés lors de l’examen. Les alertes d’analyse du code concernant vos modifications apparaissent dans la pull request afin de les corriger avant de fusionner.
-
Utiliser Copilot pour interpréter les commentaires, répondre aux questions, proposer des correctifs et résoudre les conflits de fusion avec les agents.