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

187

Une fonctionnalité ajoutée le 7 décembre 2016, annoncée sur le blog GitHub, a introduit la possibilité d'ajouter des réviseurs à une Pull Request

Option d'examen GitHub

Vous pouvez désormais demander un examen explicitement aux collaborateurs, ce qui vous permet de spécifier plus facilement qui vous souhaitez examiner votre pull request.

Vous pouvez également voir une liste des personnes dont vous attendez un examen 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és.

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

Avec les deux options désormais disponibles, quel est le rôle de chaque option puisqu'elles partagent toutes les deux le même objectif final?

Cezar Augusto
la source
1
quand "fonction de cessionnaire" libère la première fois? Y a-t-il un article qui l'introduit?
babeyh le

Réponses:

135

ÉDITER:

Après avoir discuté avec plusieurs responsables de l'OSS, les réviseurs sont définis comme ce que le mot est censé être: revoir (le code de quelqu'un) et «assigné» a une définition plus lâche expliquée ci-dessous.

Pour "réviseur" : quelqu'un dont vous souhaitez réviser 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, comme le suggère automatiquement GitHub.

Pour «cessionnaire» : à l'équipe / responsable du projet ce que cela signifie et il n'y a pas de définition stricte. Cela peut être l'ouvreur de relations publiques, ou une personne responsable de ce domaine (qui acceptera le PR une fois l'examen terminé ou le fermera simplement). Ce n'est pas à GitHub de définir ce qu'il laisse libre aux responsables de projet ce qui convient le mieux à leur projet.

Réponse précédente:

Ok, je vais répondre à ma propre question.

Pour le PR des utilisateurs avec accès en écriture: le cessionnaire serait la même personne qui a ouvert le PR, et le réviseur remplacerait l'ancienne fonction du cessionnaire (code de révision), étant celui-ci quelqu'un de son choix.

Pour les RP des utilisateurs sans accès en écriture (contributeurs externes): une personne disposant d'un accès en écriture se chargerait (ou un autre membre ayant le droit d'écriture) de réviser le PR (réviseur). Le cessionnaire est vide.

Pour les relations publiques inachevées provenant de contributeurs externes : le membre ayant accès en écriture prendrait le travail inachevé et l'attribuerait à sa place. Elle sera responsable de terminer la tâche, en tant que cessionnaire . Étant donné que la raison principale des PR est d'examiner les changements, elle sélectionnerait d'autres personnes pour examiner les changements.

Cezar Augusto
la source
24
Pour chaque nouveau membre de l'équipe, je dois envoyer un lien vers cette réponse pour expliquer comment gérer les cessionnaires et les réviseurs. Ce qui m'amène à penser que quelque chose ne va pas du tout ici :(
Andrey Kuleshov
Un cessionnaire doit-il avoir un accès en écriture?
Emre Sülün
y a-t-il une différence de comportement de notification par e-mail entre les deux?
jxramos
26

Dans GitHub, un réviseur est une personne qui examine la demande d'extraction. Un propriétaire de projet peut demander un examen à n'importe lequel 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 examinée par l'un des responsables avec un accès en écriture.

Selon la documentation officielle de github , le cessionnaire est une personne qui travaille sur des problèmes spécifiques et des pull requests. Il est parfois confus en tant que critique. Il est en fait destiné à être utilisé avec des problèmes plutôt que des pull request afin que, lorsque nous recevons un problème, nous puissions assigner quelqu'un pour le résoudre. Dans une demande d'extraction, un cessionnaire fait référence à une personne qui est en charge de fusionner cette demande d'extraction après avoir reçu des commentaires et des demandes de modification d'autres responsables.

Gautam Krishna R
la source
2
Merci pour la réponse, mais je ne pense pas que cela réponde complètement à la question. Vous pouvez attribuer un problème à quelqu'un (elle sera donc la personne responsable du problème), mais lorsque le PR est envoyé, quelqu'un sera le réviseur (cessionnaire PR), et à ce stade, je ne suis toujours pas clair sur la différence entre le cessionnaire et critique.
Cezar Augusto
14

Selon la réponse acceptée. Oui, «cessionnaire» a une définition plus vague 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 1 examinateur, qui suggère des changements et approuve finalement le PR. Pendant la phase d'examen, 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 le PR est approuvé et prêt pour le contrôle qualité ou la fusion directe, un nouveau «cessionnaire» du contrôle qualité est ajouté. De cette façon, la liste des «cessionnaires» augmente.

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

  1. Auteur de la demande d'extraction
  2. Auteur travaillant sur des suggestions de changement de relations publiques (généralement identiques à 1)
  3. Personne QA impliquée
  4. Personne responsable de la fusion (généralement identique à 2 ou 3)

L'utilisation de «cessionnaire» permet de localiser facilement le PR à 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 tout simplement author:raya-dumaspour trouver tous les articles créés par l'auteur (numéros, PR)

et d'autres requêtes similaires pour faciliter le processus de recherche. Les "jalons" sont également très utiles pour faciliter la recherche de relations publiques.

Capture d'écran Github, Q4 2017

lal
la source
Très bien expliqué.
Nitin Gaur
Il convient de mentionner que vous pouvez simplement rechercher l'auteur: my-github-handle pour trouver le PR créé par une personne
Wisienkas
1

Avant, GitHub n'avait qu'un champ d'assigné et aucun champ de réviseur. Il n'y avait pas de distinction à l'époque, donc le champ du destinataire était le plus souvent utilisé comme champ de relecteur.

Mais utilisez-les de la manière qui convient à votre projet.

Ericcurtin
la source