web-dev-qa-db-fra.com

Comment reprendre le processus de révision après avoir mis à jour la demande de pull sur GitHub?

J'ai bifurqué le dépôt github et créé une demande de tirage. Le critique m'a demandé des modifications:

Github changes requested

J'ai validé et poussé les changements nécessaires dans la branche de demande de tirage, ils s'affichent donc sur la page de demande de tirage. Maintenant, je veux demander au contributeur de continuer à revoir mes modifications.

Y a-t-il un bouton ou une action pour cela? Je ne trouve pas ça.

Buttons

19
Nikita

2 ans et demi plus tard, (février 2019), vérifiez si le nouveau bouton "Re-request review" aiderait.

Comme illustré sur Twitter , si votre PR a déjà été révisé, mais maintenant il est temps pour le tour 2: Avec de nouveaux yeux et un nouveau code, demandez à nouveau un examen ????

Re-Request a review


Note/mise en garde: (from " Demande de révision d'une demande de pull ")

Les auteurs de demandes d'extraction ne peuvent pas demander de révision à moins qu'ils ne soient propriétaires du référentiel ou collaborateurs disposant d'un accès en écriture au référentiel.

Comme indiqué dans les commentaires par gertvdijk :

Je suis un contributeur sur le projet et le propriétaire du PR.
Cela n'a pas de sens pour moi que je dois être un collaborateur sur le projet pour demander une révision?

Oh, wow, c'est par conception que les auteurs de relations publiques ne peuvent pas utiliser cette fonctionnalité?
Merde, ça tue beaucoup la fonctionnalité entière.

12
VonC

Le réviseur est retiré de la liste réelle (même s'il y est répertorié avec une croix), il ne voit même plus la demande d'extraction dans ses tâches. Ajoutez-le à nouveau comme critique.

5
Michael Bláha

Non, il n'y a aucun moyen que vous puissiez en informer un utilisateur intime.

Vous pouvez indiquer que vous avez apporté des modifications - ce commentaire lui donnera une notification.

En dehors de cela, si possible, informez-le via un autre moyen.

3
prabodhprakash

Je suis d'accord avec @prabodhprakash, utilisez le Gear à côté de "Reviewers" pour resélectionner le réviseur. Cela fait passer son statut du "x" rouge au cercle orange.

Le problème est que le statut du PR reste "Modifications demandées". Il devrait y avoir un état après cela, par exemple "Modifications résolues" ou "Examen des besoins", qui est lancé par le demandeur, via un bouton, lorsqu'il est prêt à réexaminer le PR. Sinon, le PR semble toujours dans l'état "déjà examiné et rejeté".

3
Barbituate

Je pense que vous attendez juste si le critique regarde le problème ou écrive un commentaire que vous avez fait le changement demandé (je ferais ce dernier). Ensuite, le réviseur peut avoir un autre regard et approuver le PR.

2
Vampire

La solution est de rejeter l'examen . Vous pouvez le faire en toute confiance car la conversation d'origine sera toujours présente et vous serez invité à fournir un commentaire pour vous expliquer pourquoi vous faites cela.

En résumé, recherchez la flèche déroulante à côté du "1 changement demandé". À partir de là, vous pourrez "demander une nouvelle révision", mais notez que cela ne changera pas l'état à lui seul. De plus, ou à la place, vous devrez choisir "rejeter la révision" et saisir un commentaire, tel que "effectué les modifications demandées". Une fois que vous avez confirmé, votre demande de retrait devrait s'afficher à nouveau comme "examen requis" sur la liste des RP, de sorte que la balle ne sera plus dans votre camp.

0
Nagev