Skip to main content

Approbation d’une demande de tirage comportant des revues obligatoires

Si votre dépôt nécessite des révisions, les demandes de tirage (pull request) doivent avoir un nombre spécifique de révisions d’approbation de la part de personnes disposant d’autorisations d’écriture ou d’administrateur dans le dépôt avant de pouvoir être fusionnées.

Pour plus d’informations sur les révisions requises, consultez « À propos des branches protégées ».

Vous pouvez commenter une demande de tirage, approuver les changements ou demander des améliorations avant de l’approuver. Pour plus d’informations, consultez « Révision des changements proposés dans une demande de tirage ».

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

Conseil : Si une demande de tirage que vous avez approuvée a changé de manière significative, vous pouvez masquer votre revue. La demande de tirage a besoin d’une nouvelle revue avant de pouvoir être fusionnée. Pour plus d’informations, consultez « Ignorer la révision d’une demande de tirage ».

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

  4. Passez en revue les modifications dans la demande de tirage et commentez éventuellement des lignes spécifiques ou des fichiers spécifiques. Pour plus d’informations, consultez « Révision des changements proposés dans une demande de tirage ».

  5. 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é.

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

  7. Sélectionnez Approuver pour approuver la fusion des changements proposés dans la demande de tirage.

  8. Cliquez sur Envoyer la révision.

Conseils :

  • Si 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