Approving a pull request with required reviews

Approve pull requests with required reviews, including setting approval rules, reviewing changes, and submitting feedback before merging.

You can comment on a pull request, approve the changes, or request improvements before approving. See Reviewing proposed changes in a pull request.

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

If a pull request you approved has changed significantly, you can dismiss your review. The pull request will need a new review before it can be merged. See Dismissing a pull request review.

  1. Sous le nom de votre 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. Sur la requête 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. Review the changes in the pull request. Optionally, comment on specific lines or files. See Reviewing proposed changes in a pull request.

  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. Select Approve to approve merging the proposed changes.

  8. Cliquez sur Envoyer la révision.

Conseil

  • 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.
  • Les auteurs de demandes de tirage ne peuvent pas approuver leurs propres demandes de tirage.

Further reading