Partager via


Obtenir des commentaires en utilisant des demandes de tirage

Les demandes de tirage prennent en charge la révision et la fusion du code en un seul processus collaboratif. Une fois qu’un développeur ajoute une fonctionnalité ou un correctif de bogue, il crée une demande de tirage pour commencer le processus de fusion des modifications dans la branche en amont. D’autres membres de l’équipe ont ensuite la possibilité d’examiner et d’approuver le code avant qu’il ne soit finalisé. Utilisez des demandes de tirage pour passer en revue les travaux en cours et obtenir des commentaires sur les modifications. Mais il n’y a pas d’engagement à fusionner les modifications. Un propriétaire peut abandonner une demande de tirage à tout moment.

Obtenir une révision du code

La révision du code effectuée dans le cadre d'une demande de tirage ne consiste pas à simplement trouver des bogues évidents; c'est à cela que servent les tests. Une bonne révision du code intercepte les problèmes moins évidents pouvant entraîner des problèmes coûteux plus tard.

Les révisions du code aident à protéger l’équipe contre les fusions incorrectes et les builds endommagées qui minent la productivité de l'équipe. Les révisions détectent les problèmes avant la fusion, protégeant ainsi les branches importantes des modifications indésirables.

Les révisions de code encouragent également et renforcent la collaboration et la communication entre les développeurs. Et l’équipe obtient un historique clair de toutes les modifications apportées entre la branche principale et les branches de fonctionnalités.

Croisez les savoir-faire et diffusez des stratégies de résolution des problèmes grâce à un large éventail de réviseurs dans les révisions de code. La diffusion des compétences et connaissances rend l'équipe plus forte et plus résiliente.

Émettre des commentaires utiles

Les revues de haute qualité commencent par des commentaires de haute qualité. Les principales caractéristiques des commentaires efficaces dans une demande de tirage sont les suivantes :

  • Demandez aux bonnes personnes de réviser la demande de tirage.
  • Assurez-vous que les réviseurs savent ce que fait le code.
  • Fournissez des commentaires constructifs et actionnables.
  • Répondre aux commentaires en temps opportun.

Quand vous affectez des réviseurs à une demande de tirage, assurez-vous de sélectionner le bon ensemble de réviseurs. Les réviseurs doivent savoir comment fonctionne le code, mais également inclure des développeurs travaillant dans d’autres domaines afin qu’ils puissent partager leurs idées.

Fournissez une description claire des modifications et fournissez une build du code qui a le correctif ou la fonctionnalité qui fonctionne dans celui-ci. Les réviseurs doivent faire un effort pour fournir des commentaires sur les modifications avec lesquelles ils ne sont pas d’accord. Identifiez le problème et fournissez des suggestions spécifiques sur sur ce qui pourrait être fait différemment. Ces commentaires ont un but clair et ils sont faciles à comprendre pour le propriétaire de la demande de tirage.

Le propriétaire de la demande de tirage doit répondre aux commentaires, accepter des suggestions ou expliquer pourquoi ils refusent de les appliquer. Certaines suggestions sont bonnes, mais peuvent être en dehors de l’étendue de la demande de tirage. Prenez ces suggestions et créez des éléments de travail et branches de fonctionnalité distincts de la demande de tirage pour apporter ces modifications.

Protéger les branches avec des stratégies

Il y a quelques branches critiques dans un dépôt sur lesquelles les équipes comptent pour être toujours en bonne forme, comme la main branche. Les équipes peuvent exiger des demandes de tirage pour apporter des modifications à ces branches avec des plateformes telles que GitHub et Azure DevOps. Les développeurs qui envoient (push) des modifications directement vers ces branches protégées voient leurs envois rejetés.

Ajoutez des conditions supplémentaires à vos demandes de tirage pour appliquer un niveau de qualité de code plus élevé dans les branches clés. Une build propre du code fusionné et l’approbation de plusieurs réviseurs sont des exigences supplémentaires souvent utilisées pour protéger les branches clés.

En savoir plus

GitHub dispose d’une documentation complète sur la façon de proposer des modifications à votre travail avec des demandes de tirage.

En savoir plus sur l’envoi de commentaires importants dans les révisions et l’utilisation de modèles de demande de tirage pour fournir des conseils à vos réviseurs. Azure DevOps offre également une expérience de demande de tirage riche qui est facile à utiliser et à mettre à l’échelle en fonction des besoins.