Obtenez des commentaires avec des pull requests

Les pull requests prennent en charge l'examen et la fusion du code dans un processus unique de collaboration. Une fois qu’un développeur ajoute une fonctionnalité ou un correctif de bogue, il crée une demande de pull pour commencer le processus de fusion des modifications dans la branche principale. 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. Le propriétaire peut abandonner une pull request à tout moment.

Faites réviser votre code

La révision du code effectuée dans le cadre d'une pull request n'est pas seulement pour trouver des bogues évidents ; c'est à cela que servent les tests. Une bonne révision du code intercepte des problèmes moins évidents qui pourraient entraîner des problèmes coûteux ultérieurement.

Les révisions de code aident à protéger l’équipe contre les fusions incorrectes et les builds rompues qui sapent la productivité de l’équipe. Examine les problèmes rencontrés avant la fusion, protégeant les branches importantes contre les 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.

Inter-polliniser les expertises et diffuser les stratégies de résolution des problèmes en utilisant un large éventail de réviseurs dans les revues de code. La diffusion des compétences et des connaissances rend l’équipe plus forte et plus résiliente.

Envoyez de bons commentaires

Les revues de haute qualité commencent par des commentaires de haute qualité. Les clés des excellents retours dans une pull request sont les suivantes :

• Faites examiner la pull request par les bonnes personnes.
• Assurez-vous que les réviseurs savent ce que fait le code.
• Donnez des commentaires exploitables et constructifs.
• Répondre aux commentaires en temps opportun.

Lorsque vous attribuez des réviseurs à une pull request, veillez à 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 qu’ils n’acceptent pas. Identifiez le problème et donnez des suggestions spécifiques sur ce qui pourrait être fait différemment. Ces commentaires ont une intention claire et sont faciles à comprendre pour le propriétaire du pull request.

Le propriétaire du pull request doit répondre aux commentaires, accepter les suggestions ou expliquer pourquoi il refuse de les appliquer. Certaines suggestions sont bonnes, mais peuvent être hors du champ de la requête de tirage. Prenez ces suggestions et créez des éléments de travail et des branches de fonctionnalités séparés de la pull request pour effectuer ces modifications.

Protéger les branches avec des stratégies

Il existe quelques branches critiques dans un dépôt sur lesquelles les équipes comptent toujours pour être en bon état, comme la branche main. Les équipes peuvent exiger des pull requests pour apporter des modifications à ces branches avec des plateformes telles que GitHub et Azure DevOps. Les développeurs qui poussent directement les modifications vers les branches protégées auront leurs push rejetés.

Ajoutez des conditions supplémentaires aux requêtes d'extraction pour garantir une meilleure qualité de code dans des 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.

Learn more

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

En savoir plus sur donner des retours constructifs dans les révisions de code et l’utilisation de modèles de pull request pour fournir des conseils à vos réviseurs. Azure DevOps offre également une expérience de pull request enrichissante qui est facile à utiliser et s'adapte en fonction des besoins.