Sur GitHub, quelle est la différence entre examinateur et cessionnaire?
Une fonctionnalité ajoutée le 7 décembre 2016, annoncée sur le blog de GitHub, introduit l'option pour ajouter des réviseurs à une demande de tirage
Vous pouvez désormais demander explicitement une révision à des collaborateurs, ce qui vous permet de spécifier plus facilement les personnes que vous souhaitez consulter.
Vous pouvez également voir une liste des personnes pour lesquelles vous attendez une révision dans la barre latérale de la page de demande de retrait, ainsi que le statut des critiques de ceux qui les ont déjà laissées.
Cependant, la définition explicite d'un relecteur pour un PR a déjà été effectuée en affectant des personnes (option assignées ).
Avec les deux options maintenant disponibles, quel est le rôle de chaque option puisqu'elles partagent le même objectif final?
EDIT:
Après avoir discuté avec plusieurs responsables de logiciels libres, les réviseurs sont définis comme ce que le mot est supposé être: réviser (le code de quelqu'un) et "l'assigné" a une définition plus lâche. expliqué ci-dessous.
Pour "relecteur": quelqu'un à qui vous voulez relire le code. Pas nécessairement la personne responsable de cette zone ou responsable de la fusion du commit. Peut être quelqu'un qui a déjà travaillé sur ce morceau de code auparavant, comme le suggère automatiquement GitHub.
Pour "assigné": ce que cela signifie et ce n'est pas une définition stricte. Il peut s’agir de l’ouvreur des relations publiques ou d’une personne responsable de ce domaine (qui va accepter les relations publiques une fois l’examen terminé ou tout simplement le fermer). Ce n'est pas à GitHub de définir ce qui le laisse ouvert aux responsables du projet ce qui convient le mieux à leur projet.
Réponse précédente:
Ok je vais aller de l'avant et répondre à ma propre question.
Pour les RP des utilisateurs ayant un accès en écriture:, le cessionnaire serait la même personne que celle qui l'a ouvert, et le réviseur remplacerait l'ancienne fonction de cessionnaire (code de révision), étant celle du choix du cessionnaire.
Pour les RP des utilisateurs sans accès en écriture (contributeurs extérieurs): Une personne ayant un accès en écriture s'auto-attribuerait (ou un autre membre privilégié en écriture) pour examiner le PR (Relecteur). Le cessionnaire est vide.
Pour les relations publiques inachevées provenant de contributeurs extérieurs: le membre avec accès en écriture prend le travail inachevé et l'assigne pour elle. Elle sera responsable de l’achèvement de la tâche, en tant que destinataire. Puisque la raison principale des RP est de réviser les changements, elle choisissait d'autres personnes pour examiner les changements.
Dans GitHub, un critique est une personne qui examine la demande d'extraction. Un propriétaire de projet peut demander une révision à l'un des responsables. Ils peuvent même définir une option pour que la demande d'extraction ne puisse être fusionnée que si elle est vérifiée par l'un des responsables disposant d'un accès en écriture.
Selon le document officiel documentation github , le cessionnaire est une personne qui travaille sur des problèmes spécifiques et sur les demandes d'extraction. Il est parfois confus en tant que critique. En réalité, il est conçu pour être utilisé avec des problèmes plutôt que comme une demande de retrait, de sorte que lorsque nous recevons un problème, nous puissions assigner à quelqu'un le soin de le résoudre. Dans une demande de tirage, un cessionnaire fait référence à une personne en charge de la fusion de cette demande de tirage après avoir reçu des commentaires et des demandes de modification d'autres mainteneurs.
Selon la réponse acceptée. Oui, le terme "cessionnaire" a une définition plus souple et peut être utilisé différemment pour répondre aux besoins des équipes.
Dans notre équipe de 8 développeurs, dans la plupart des PR, nous avons un critique qui suggère des modifications et approuve finalement le PR. Pendant la phase de révision, "cessionnaire" est la personne qui a ouvert le récipiendaire principal; plus tard, si PR est repris par un autre développeur, un nouveau "destinataire" est ajouté. Une fois que PR est approuvé et prêt pour le contrôle qualité ou la fusion directe, un nouveau "responsable" du contrôle qualité est ajouté. De cette façon, la liste des "destinataires" s'allonge.
Nous utilisons "cessionnaire" pour désigner collectivement les personnes suivantes:
- Pull Request Auteur
- Auteur travaillant sur des suggestions de changement de relations publiques (généralement identiques à 1)
- QA personne impliquée
- Personne responsable de la fusion (généralement identique à 2 ou 3)
L'utilisation de "cessionnaire" aide à localiser facilement le RP à l'avenir. Un de mes projets compte plus de 3000 PR.
is:open is:pr author:raya-dumas
is:closed is:pr assignee:raya-dumas
Ou simplement author:raya-dumas pour trouver tous les éléments créés par l'auteur (numéros, relations publiques)
et d'autres requêtes similaires pour faciliter le processus de recherche. Les "jalons" sont également utiles pour faciliter la recherche de relations publiques.
