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

Question

Une fonction qui a été ajoutée sur Déc, 7, 2016, a annoncé sur GitHub blog, a introduit l'option pour ajouter des réviseurs d'une Pull Request

Vous pouvez maintenant demander un examen explicite de ses collaborateurs, le rendant plus facile de préciser à qui vous voulez revoir votre pull request.

Vous pouvez également voir une liste de personnes qui vous sont en attente d'examen de la demande d'extraction barre latérale de la page, ainsi que le statut de commentaires de ceux qui ont déjà quitté.

Cependant, la définition explicite d'un réviseur pour une PR a été déjà fait par l'affectation de personnes (les cessionnaires option).

Avec les deux options maintenant disponibles, quel est le rôle de chaque option, car ils partagent tous deux le même objectif final?

Answer 1

EDIT:

Après avoir discuté avec plusieurs OSS responsables, les évaluateurs est définie comme ce que le mot censé être: examiner (quelqu'un de code) et "cessionnaire" a un looser définition expliqué ci-dessous.

Pour le "réviseur": la personne que vous voulez vérifier le code. Pas nécessairement la personne responsable de cette zone ou de la responsable de la fusion de la commettre. Peut être quelqu'un qui a travaillé sur ce morceau de code avant, comme GitHub auto-suggère.

Pour le "cessionnaire": jusqu'à l'équipe du projet/responsable de ce que cela signifie et il n'y a pas de définition stricte. Il peut être le PR ouvreur, ou à une personne responsable pour cette zone (qui va accepter la PR après l'examen est fait ou juste la fermer). Ce n'est pas à GitHub pour définir ce que c'est en la laissant ouverte pour les responsables de projet ce qui convient le mieux à leur projet.

La réponse précédente:

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

Pour le PR des utilisateurs avec l'accès en écriture: le Cessionnaire est la même personne qui a ouvert le PR, et le critique de remplacer l'ancien cessionnaire fonction (revue de code), étant l'un quelqu'un de cessionnaire choix.

Pour le PR des utilisateurs sans accès en écriture (contributeurs extérieurs): Quelqu'un accès en écriture à attribuer, pour elle-même (ou d'une autre écriture privilège membre), pour l'examen de la PR (Examinateur). Le cessionnaire est vide.

Pour inachevée PR de contributeurs extérieurs: l'accès en écriture membre de prendre le travail inachevé et affecter pour elle. Elle sera responsable de la finition de la tâche, étant le Cessionnaire. Depuis la principale raison de PRs est d'examiner les changements apportés, elle serait de sélectionner quelques autres personnes pour examiner les changements.

Answer 2

Dans GitHub un réviseur est une personne qui examine la demande d'extraction. Un porteur de projet peut demander un réexamen à partir de l'responsables, Ils peuvent même définir une option de sorte que la demande d'extraction peuvent être regroupés que si elle est examinée par l'un des responsable avec accès en écriture.

Selon github de la documentation, le Cessionnaire est une personne qui travaille sur des questions spécifiques et des pull requests. Il est parfois confondu comme un examinateur. Il est fait pour être utilisé avec des questions plutôt que de tirer la demande, de sorte que lorsque nous recevons une question que nous pouvons attribuer à quelqu'un pour le réparer. Dans une pull request, un cessionnaire se réfère à une personne qui est en charge de la fusion qui tirent la demande, d'obtenir des commentaires et des demandes de changement à partir d'autres responsables.

Answer 3

Selon accepté de répondre. Oui, le "cessionnaire" a un looser définition et peuvent être utilisés de différentes manières pour répondre à une des équipes de besoin.

Dans notre équipe de 8 développeurs, dans la plupart des PRs nous avons 1 critique, qui suggère des changements et approuve finalement le PR. Lors de l'examen de la phase, "cessionnaire" est la personne qui a ouvert le PR; plus tard, si le PR est repris par d'autres développeurs, un nouveau "cessionnaire" est ajouté. Une fois que PR est approuvé et prêt pour l'assurance qualité ou de la fusion direct, une nouvelle QA "cessionnaire" est ajouté. De cette façon, "cessionnaire" liste grandit.

Nous utilisons des "cessionnaire" pour désigner les personnes suivantes collectivement:

1. Pull Request Auteur
2. Auteur de travail sur la PR des propositions de modification (généralement le même que le 1)
3. QA personne impliquée
4. La personne responsable de fusion (généralement de même que 2 ou 3)

À l'aide de "cessionnaire" aide à la localisation de la PR à l'avenir facilement. L'un de mes projet a >3000 PRs.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Ou juste author:raya-dumas pour trouver tous les éléments de l'auteur créé (questions, PRs)

et d'autres semblables, les requêtes pour faciliter le processus de recherche. "étapes" sont très utiles à utiliser aussi bien à l'aise PR de recherche.