Sur GitHub, quelle est la différence entre le réviseur et le cessionnaire?

Une fonctionnalité ajoutée le 7 décembre 2016, annoncée sur le blog GitHub, a introduit l’option d’append des réviseurs à une requête de tirage.

Option de révision GitHub

Vous pouvez maintenant demander une révision explicite auprès des collaborateurs, ce qui facilite la spécification de qui vous souhaitez examiner votre demande d’extraction.

Vous pouvez également voir une liste de personnes en attente de révision dans la barre latérale de la page de demande d’extraction, ainsi que le statut des avis de ceux qui les ont déjà quitté.

Cependant, la définition explicite d’un réviseur pour un PR était déjà effectuée en assignant des personnes (option des destinataires ).

Avec les deux options maintenant disponibles, quel est le rôle de chaque option puisqu’elles partagent le même objective final?

MODIFIER:

Après avoir discuté avec plusieurs responsables OSS, les réviseurs sont définis comme étant ce que le mot est censé être: réviser (code de quelqu’un) et “cessionnaire” a une définition plus souple expliquée ci-dessous.

Pour “réviseur” : quelqu’un que vous souhaitez revoir le code. Pas nécessairement la personne responsable de cette région ou responsable de la fusion du commit. Peut-être quelqu’un qui a déjà travaillé sur ce morceau de code, comme le suggère automatiquement GitHub.

Pour “cessionnaire” : jusqu’à l’équipe / le responsable du projet, ce que cela signifie et aucune définition ssortingcte. Il peut s’agir de l’ouvreur de PR ou de quelqu’un responsable de cette zone (qui va accepter le PR après la révision ou la fermer). Ce n’est pas à GitHub de définir ce qu’il laisse aux responsables de projets ce qui leur convient le mieux.

Réponse précédente:

Ok je vais aller de l’avant et répondre à ma propre question.

Pour PR des utilisateurs avec access en écriture: le destinataire est la même personne qui a ouvert le PR, et le réviseur remplacera l’ancienne fonction de destinataire (code de révision), en tant que personne du choix du destinataire.

Pour les PR des utilisateurs sans access en écriture (consortingbuteurs externes): une personne ayant un access en écriture s’atsortingbuerait elle-même (ou un autre membre privilégiant l’écriture), pour examiner le PR (réviseur). Le destinataire est vide.

Pour les RP non finis provenant de consortingbuteurs externes : le membre ayant access en écriture prendrait le travail inachevé et l’atsortingbuerait. Elle sera chargée de terminer la tâche en tant que cessionnaire . Étant donné que la raison principale des relations publiques est d’examiner les changements, elle sélectionnerait d’autres personnes pour examiner les changements.

Dans GitHub, un relecteur est une personne qui examine la demande de tirage. Un propriétaire de projet peut demander une révision à l’un des responsables. Il peut même définir une option pour que la requête d’extraction puisse être fusionnée uniquement si elle est examinée par l’un des responsables avec un access en écriture.

Selon la documentation officielle de github , le cessionnaire est une personne qui travaille sur des problèmes spécifiques et tire des requêtes. Il est parfois confus en tant que critique. Il est en fait destiné à être utilisé avec des problèmes plutôt que de tirer une requête, de sorte que lorsque nous recevons un problème, nous pouvons affecter quelqu’un pour le résoudre. Dans une demande d’extraction, un destinataire fait référence à une personne chargée de fusionner cette requête d’extraction après avoir reçu des commentaires et modifié les demandes d’autres responsables.

Selon la réponse acceptée. Oui, le “cessionnaire” a une définition plus souple et peut être utilisé différemment pour répondre aux besoins d’une équipe.

Dans notre équipe de 8 développeurs, dans la plupart des PR, nous avons un réviseur qui suggère des changements et approuve le PR. Au cours de la phase de révision, le “cessionnaire” est la personne qui a ouvert le PR; plus tard, si PR est repris par un autre développeur, un nouveau “cessionnaire” est ajouté. Une fois que PR est approuvé et prêt pour l’assurance qualité ou la fusion directe, un nouveau “responsable” de l’assurance qualité est ajouté. De cette façon, la liste “cessionnaire” augmente.

Nous utilisons «cessionnaire» pour désigner collectivement les personnes suivantes:

  1. Pull Request Author
  2. Auteur travaillant sur des suggestions de changement de RP (généralement identique à 1)
  3. QA personne impliquée
  4. Personne responsable de la fusion (généralement identique à 2 ou 3)

Utiliser “cessionnaire” permet de localiser facilement le PR à l’avenir. Un de mes projets a> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

et d’autres requêtes similaires pour faciliter le processus de recherche. Les “jalons” sont très utiles pour faciliter la recherche de relations publiques.

Screenshot Github, T4 2017