Révision des changements proposés dans une demande de tirage

Dans une demande de tirage (pull request), vous pouvez examiner et discuter des commits, des fichiers modifiés et des différences (ou « diff ») entre les fichiers dans la base et comparer les branches.

Tool navigation

Dans cet article

À propos de la révision des demandes de tirage

Vous pouvez réviser des changements dans une demande de tirage, fichier par fichier. Quand vous révisez des fichiers dans une demande de tirage, vous pouvez laisser des commentaires individuels sur des changements spécifiques. Une fois que vous avez terminé de réviser un fichier, vous pouvez le marquer comme consulté. Le fichier est alors réduit, ce qui vous permet d'identifier facilement ceux qu'il vous reste à réviser. Une barre de progression située dans l'en-tête de la demande de tirage indique le nombre de fichiers que vous avez consultés. Une fois que vous avez révisé le nombre de fichiers souhaité, vous pouvez approuver la demande de tirage ou demander des changements supplémentaires en envoyant votre révision avec un commentaire récapitulatif.

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 ».

Démarrage d'une révision

  1. Sous le nom du référentiel, cliquez sur Demandes de tirage.

    Capture d’écran de la page principale d’un dépôt. Dans la barre de navigation horizontale, un onglet intitulé « Demandes de tirage » est indiqué en orange foncé.

  2. Dans la liste des demandes de tirage, cliquez sur la demande de tirage que vous voulez réviser.

  3. Dans la demande de tirage, cliquez sur Fichiers modifiés.

    Capture d’écran des onglets d’une demande de tirage. L’onglet « Fichiers modifiés » est indiqué en orange foncé.
    Vous pouvez modifier l'aspect de la vue des différences sous cet onglet en cliquant sur et en choisissant la vue unifiée ou fractionnée. Ce choix s'applique quand vous affichez les différences pour d'autres demandes de tirage.

    Capture d'écran de l'onglet « Fichiers changés » d'une demande de tirage. Le menu « Vue Diff » est indiqué en orange foncé.

    Vous pouvez aussi choisir de masquer les différences d'espaces blancs. Ce choix s'applique uniquement à cette demande de tirage et reste le même pour votre prochaine consultation de cette page.

  4. Filtrez éventuellement les fichiers pour voir uniquement ceux que vous voulez réviser ou utilisez l'arborescence des fichiers pour accéder à un fichier spécifique. Pour plus d’informations, consultez « Filtrage des fichiers dans une demande de tirage ».

  5. Pointez sur la ligne de code dans laquelle vous souhaitez ajouter un commentaire, puis cliquez sur l’icône de commentaire bleue.

    Capture d’écran d’une différence dans une demande de tirage. En regard d’un numéro de ligne, une icône plus bleue est mise en évidence avec un contour orange.

  6. Si vous le souhaitez, vous pouvez ajouter un commentaire sur plusieurs lignes. Vous pouvez cliquer sur le numéro de ligne de la première ligne sur laquelle vous souhaitez commenter et faire glisser vers le bas pour sélectionner une plage de lignes, puis cliquer sur l’icône de commentaire bleue sur la dernière ligne sur laquelle vous souhaitez commenter. Vous pouvez également cliquer sur l’icône de commentaire bleue en regard de la première ligne sur laquelle vous souhaitez commenter, puis faire glisser vers le bas jusqu’à la dernière ligne sur laquelle vous souhaitez commenter.

  7. Entrez votre commentaire dans le champ de commentaire.

  8. Si vous voulez suggérer un changement spécifique pour une ou plusieurs lignes, cliquez sur , puis modifiez le texte dans le bloc de suggestion.

    Capture d’écran d’une zone de révision de commentaire. L’icône de diff de fichier pour suggérer un changement spécifique est indiquée en orange foncé.

  9. Pour commenter directement un fichier, à droite du fichier, cliquez sur et tapez votre commentaire.

    Capture d’écran d’un fichier image dans la page « Fichiers modifiés » d’une demande de tirage. À droite du fichier image, une icône de commentaire est mise en évidence avec un encadré orange.

  10. Quand vous avez terminé, cliquez sur Démarrer une révision. Si vous avez déjà démarré une révision, vous pouvez cliquer sur Ajouter un commentaire de révision.

Avant de soumettre votre révision, vos commentaires de ligne sont en attente et uniquement visibles par vous. Vous pouvez modifier des commentaires en attente à tout moment avant de soumettre votre révision. Pour annuler une révision en attente, y compris tous ses commentaires en attente, cliquez sur Réviser les changements au-dessus du code changé, puis sur Abandonner la révision.

Capture d'écran du champ de commentaire d'une révision. Le bouton « Abandonner la révision » est indiqué en orange foncé.

Révision des changements de dépendances

Si la demande de tirage contient des changements apportés aux dépendances, vous pouvez utiliser la révision des dépendances d'un fichier manifeste ou d'un fichier de verrouillage pour voir ce qui a changé et vérifier si les changements introduisent des vulnérabilités de sécurité. Pour plus d’informations, consultez « Révision des changements de dépendances dans une demande de tirage ».

  1. Dans la demande de tirage, cliquez sur Fichiers modifiés.

    Capture d’écran des onglets d’une demande de tirage. L’onglet « Fichiers modifiés » est indiqué en orange foncé.

  2. À droite de l'en-tête d'un fichier manifeste ou d'un fichier de verrouillage, affichez la révision des dépendances en cliquant sur le bouton des différences enrichies .

    Capture d'écran de l'onglet « Fichiers changés » d'une demande de tirage. Le bouton permettant d'afficher le diff enrichi, étiqueté avec une icône de fichier, est indiqué en orange foncé.

  3. Vous pouvez également examiner le diff source, car il peut y avoir des changements dans le manifeste ou le fichier de verrouillage qui ne modifient pas les dépendances, ou il peut y avoir des dépendances que GitHub ne peut pas analyser et qui, par conséquent, n’apparaissent pas dans l’examen des dépendances.

    Pour revenir à la vue du diff source, cliquez sur le bouton .

    Capture d’écran de l’onglet « Fichiers changés » d’une demande de tirage. Le bouton permettant d’afficher le diff source, qui est étiqueté avec une icône de code, est indiqué en orange foncé.

Marquage d'un fichier comme consulté

Une fois que vous avez terminé de réviser un fichier, vous pouvez le marquer comme consulté. Il est alors réduit. Si le fichier change après que vous l'avez consulté, son marquage comme consulté est annulé.

  1. Dans la demande de tirage, cliquez sur Fichiers modifiés.

    Capture d’écran des onglets d’une demande de tirage. L’onglet « Fichiers modifiés » est indiqué en orange foncé.

  2. À droite de l'en-tête du fichier que vous avez terminé de réviser, sélectionnez Consulté.

    Capture d'écran de l'en-tête d'un fichier. L'option « Consultés » est signalée en orange foncé.

Soumission de votre révision

Une fois que vous avez terminé de réviser tous les fichiers souhaités dans la demande de tirage, soumettez votre révision.

  1. Dans la demande de tirage, cliquez sur Fichiers modifiés.

    Capture d’écran des onglets d’une demande de tirage. L’onglet « Fichiers modifiés » est indiqué en orange foncé.

  2. Au-dessus du code modifié, cliquez sur Vérifier les modifications.

    Capture d’écran de l’onglet « Fichiers changés » d’une demande de tirage. Le bouton « Réviser les changements » est indiqué en orange foncé.

  3. Tapez un commentaire résumant vos commentaires sur les modifications proposées.

  4. Sélectionnez le type de révision que vous voulez laisser :

    • Sélectionnez Commenter pour laisser des commentaires généraux sans approuver explicitement les changements ni demander des changements supplémentaires.
    • Sélectionnez Approuver pour soumettre vos commentaires et approuver la fusion des changements proposés dans la demande de tirage.
    • Sélectionnez Demander des changements pour soumettre des commentaires à traiter avant de pouvoir fusionner la demande de tirage.

  5. Cliquez sur Envoyer la révision.

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.

Pour aller plus loin