Skip to main content

À propos des révisions de demande de tirage

Les révisions permettent aux collaborateurs de commenter les modifications proposées dans les demandes de tirage (pull request), d’approuver les modifications ou de demander d’autres modifications avant la fusion de la demande de tirage. Les administrateurs de dépôt peuvent exiger que toutes les demandes de tirage soient approuvées avant d’être fusionnées.

À propos des révisions de demande de tirage

Une fois une demande de tirage ouverte, toute personne disposant d’un accès en lecture peut réviser et commenter les modifications qu’elle propose. Vous pouvez également suggérer des modifications spécifiques de lignes de code, que l’auteur peut appliquer directement à partir de la demande de tirage. Pour plus d’informations, consultez « Révision des changements proposés dans une demande de tirage ».

Par défaut, dans les dépôts publics, tous les utilisateurs peuvent envoyer des révisions qui approuvent ou demandent des changements sur une demande de tirage. Les propriétaires d’organisation et les administrateurs de référentiel peuvent limiter les personnes autorisées à approuver des révisions de demandes de tirage ou des modifications de demandes. Pour plus d’informations, consultez « Gestion des révisions de demande de tirage au sein de votre organisation » et « Gestion des révisions de demande de tirage au sein de votre référentiel ».

Les propriétaires et collaborateurs du dépôt peuvent demander une révision de demande de tirage d’une personne spécifique. Des membres de l’organisation peuvent également demander une révision de demande de tirage d’une équipe disposant d’un accès en lecture au dépôt. Pour plus d’informations, consultez « Demande d’une évaluation de demande de tirage ». Vous pouvez spécifier un sous-ensemble de membres de l’équipe à saisir automatiquement à la place de l’équipe toute entière. Pour plus d’informations, consultez « Gestion des paramètres de révision du code pour l’équipe ».

Les révisions permettent de discuter des modifications proposées et de s’assurer que celles-ci sont conformes aux directives de contribution du dépôt, ainsi qu’à d’autres normes de qualité. Vous pouvez définir quelles personnes ou équipes possèdent certains types ou zones de code dans un fichier CODEOWNERS. Quand une demande de tirage modifie du code ayant un propriétaire défini, cette personne ou équipe sera automatiquement invitée à intervenir en tant que réviseur. Pour plus d’informations, consultez « À propos des propriétaires de code ».

Pour une introduction à la demande et à la fourniture de révisions de demande de tirage, consultez le cours Examinez les demandes de tirage GitHub Skills.

Vous pouvez planifier des rappels pour les demandes de tirage à réviser. Pour plus d’informations, consultez « Gestion des rappels planifiés pour votre équipe ».

Une révision a trois statuts possibles :

  • Commenter : formuler des commentaires généraux sans approuver explicitement les modifications ou demander des modifications supplémentaires.
  • Approuver : formuler des commentaires et approuver la fusion des modifications proposées dans la demande de tirage.
  • Demander des modifications : formuler des commentaires qui doivent être traités avant que la demande de tirage puisse être fusionnée.

Tip

  • L’option Demander des changements est purement informationnelle et n’empêche pas la fusion, sauf si un ensemble de règles ou une règle de protection de branche classique est configuré avec l’option « Exiger une demande de tirage ». Si configuré et qu’un collaborateur disposant d’un accès admin, owner ou write au référentiel envoie une révision demandant des modifications, la demande de tirage ne peut pas être fusionnée tant que ce collaborateur n’a pas soumis une autre révision qui approuve les modifications dans la demande de tirage.
  • Les propriétaires et administrateurs du dépôt peuvent fusionner une demande de tirage, même si celle-ci n’a pas reçu de révision d’approbation, ou si un réviseur qui a demandé des modifications a quitté l’organisation ou n’est pas disponible.
  • Si les révisions requises et l’abandon des révisions obsolètes sont activés et qu’un commit de modification du code est poussé vers la branche d’une demande de tirage approuvée, l’approbation est rejetée. La demande de tirage doit être de nouveau révisée et approuvée avant de pouvoir être fusionnée.
  • Lorsque plusieurs demandes de tirage ouvertes ont chacune une branche de tête (head) pointant vers le même commit, vous ne pouvez pas les fusionner si une ou les deux ont une révision en attente ou rejetée.
  • Si votre dépôt demande l’approbation des révisions des personnes avec des autorisations d’écriture ou d’administrateur, toutes les approbations des personnes avec ces autorisations sont marquées d’une coche verte, tandis que les approbations des personnes sans ces autorisations sont marquées d’une coche grise. Les approbations avec une coche grise n’affectent pas si la demande de tirage peut être fusionnée ou non.
  • Les auteurs de demandes de tirage ne peuvent pas approuver leurs propres demandes de tirage.

Vous pouvez afficher toutes les révisions qu’une demande de tirage a reçues dans la chronologie de la conversation, ainsi que les révisions par les propriétaires et collaborateurs du dépôt dans la zone de fusion de la demande de tirage.

Capture d’écran de la zone de fusion d’une demande de tirage. Une révision par Octocat avec les modifications demandées est répertoriée.

Tip

Vous pouvez trouver une demande de tirage où vous ou une équipe dont vous êtes membre êtes demandé pour une révision avec le qualificateur de recherche review-requested:[USERNAME] ou team-review-requested:[TEAMNAME]. Pour plus d’informations, consultez « Recherche de problèmes et de demandes de tirage ».

Résolution des conversations

Vous pouvez résoudre une conversation dans une demande de tirage si vous avez ouverte celle-ci ou si vous avez un accès en écriture au dépôt où la demande de tirage a été ouverte.

Pour indiquer qu’une conversation sous l’onglet Fichiers modifiés est terminée, cliquez sur Résoudre la conversation.

Toute la conversation sera réduite et marquée comme résolue, ce qui facilite la recherche de conversations qui doivent encore être traitées.

Si la suggestion dans un commentaire sort du cadre de votre demande de tirage, vous pouvez ouvrir un nouveau problème qui suit les commentaires et renvoie au commentaire d’origine. Pour plus d’informations, consultez « Création d’un problème ».

Découverte et navigation dans les conversations

Vous pouvez découvrir et accéder à toutes les conversations de votre demande de tirage à l’aide du menu Conversations affiché en haut de l’onglet Fichiers modifiés .

Dans cette vue, vous pouvez voir quelles conversations ne sont pas résolues, résolues et obsolètes. Cela facilite la découverte et la résolution des conversations.

Capture d’écran du menu « Conversations » sous l’onglet « Fichiers changés » d’une demande de tirage.

Re-demander une révision

Vous pouvez redemander une révision, par exemple, après avoir fait des changements substantiels sur votre demande de tirage. Pour demander une nouvelle révision auprès d’un réviseur, dans la barre latérale de l’onglet Conversation, cliquez sur l’icône .

Révisions requises

Les administrateurs de dépôts ou les rôles personnalisés avec l’autorisation « modifier les règles de dépôt » peuvent exiger que toutes les demandes de tirage reçoivent un nombre spécifique de révisions d’approbation avant qu’une personne ne fusionne la demande de tirage dans une branche protégée. Vous pouvez demander des révisions d’approbation à des personnes qui ont des autorisations d’écriture sur le dépôt ou à un propriétaire de code désigné. Pour plus d’informations, consultez « À propos des branches protégées ».

Astuce : si nécessaire, une personne disposant d’un accès en administration ou en écriture à un dépôt peut ignorer une révision de demande de tirage. Pour plus d’informations, consultez « Ignorer la révision d’une demande de tirage ».

Pour aller plus loin