En GitHub, ¿cuál es la diferencia entre revisor y cesionario?

187

Una característica agregada el 7 de diciembre de 2016, anunciada en el blog de GitHub, introdujo la opción de agregar revisores a una solicitud de extracción

Opción de revisión de GitHub

Ahora puede solicitar una revisión explícita de los colaboradores, lo que facilita la especificación de a quién le gustaría revisar su solicitud de extracción.

También puede ver una lista de personas de las que está esperando una revisión en la barra lateral de la página de solicitud de extracción, así como el estado de las revisiones de aquellos que ya las han dejado.

Sin embargo, la configuración explícita de un revisor para un RP ya se realizó mediante la asignación de personas ( opción de asignatarios ).

Con ambas opciones ahora disponibles, ¿cuál es el papel de cada opción ya que ambas comparten el mismo objetivo final?

Cezar Augusto
fuente
1
cuando la "función de cesionario" se libera por primera vez? ¿Hay algún artículo que lo presente?
Babeyh

Respuestas:

135

EDITAR:

Después de discutir con varios mantenedores de OSS, los revisores se definen como lo que se supone que es la palabra: revisar (código de alguien) y "cesionario" tiene una definición más flexible que se explica a continuación.

Para "revisor" : alguien que desea revisar el código. No necesariamente la persona responsable de esa área o responsable de fusionar el compromiso. Puede ser alguien que trabajó en ese fragmento de código antes, como sugiere automáticamente GitHub.

Para "cesionario" : hasta el equipo / responsable del proyecto lo que significa y no existe una definición estricta. Puede ser el abridor de relaciones públicas, o alguien responsable de esa área (que va a aceptar las relaciones públicas después de que se realice la revisión o simplemente cerrarla). No depende de GitHub definir qué es lo que deja abierto para los encargados del mantenimiento del proyecto lo que mejor se adapta a su proyecto.

Respuesta anterior:

Ok, seguiré y responderé mi propia pregunta.

Para el RP de los usuarios con acceso de escritura: el cesionario sería la misma persona que abrió el RP, y el revisor reemplazaría la antigua función del cesionario (código de revisión), siendo esta la persona elegida por el cesionario.

Para PR de usuarios sin acceso de escritura (contribuyentes externos): Alguien con acceso de escritura se asignaría a sí misma (u otro miembro con privilegios de escritura), para revisar el PR (Revisor). El cesionario está en blanco.

Para relaciones públicas inacabadas de colaboradores externos : el miembro de acceso de escritura tomaría el trabajo inacabado y se lo asignaría. Ella será responsable de terminar la tarea, siendo la Asignada . Dado que la razón principal de las relaciones públicas es revisar los cambios, seleccionaría a otras personas para revisar los cambios.

Cezar Augusto
fuente
24
Para cada nuevo miembro del equipo, debería enviar un enlace a esta respuesta para explicar cómo tratar con los asignados y los revisores. Lo que me lleva a pensar que algo está fundamentalmente mal aquí :(
Andrey Kuleshov
¿Un cesionario tiene que tener acceso de escritura?
Emre Sülün
¿Hay alguna diferencia en el comportamiento de notificación por correo electrónico entre los dos?
jxramos
26

En GitHub, un revisor es una persona que revisa la solicitud de extracción. El propietario de un proyecto puede solicitar una revisión de cualquiera de los encargados del mantenimiento. Incluso pueden establecer una opción para que la solicitud de extracción solo pueda fusionarse si es revisada por uno de los encargados del mantenimiento con acceso de escritura.

De acuerdo con la documentación oficial de Github , Assignee es una persona que está trabajando en problemas específicos y solicitudes de extracción. A veces se confunde como revisor. En realidad, está destinado a ser utilizado con problemas en lugar de solicitud de extracción para que cuando recibamos un problema podamos asignar a alguien para que lo solucione. En una solicitud de extracción, un cesionario se refiere a una persona que está a cargo de fusionar esa solicitud de extracción después de recibir comentarios y solicitudes de cambio de otros mantenedores.

Gautam Krishna R
fuente
2
Gracias por la respuesta, pero no creo que aborde la pregunta por completo. Puede asignar un problema a alguien (para que ella sea el cesionario del problema), pero cuando se envíe el RP alguien será el revisor (el cesionario del RP), y en este punto, todavía no tengo clara la diferencia entre el cesionario y crítico.
Cezar Augusto
14

Según la respuesta aceptada. Sí, "cesionario" tiene una definición más flexible y puede usarse de manera diferente para satisfacer las necesidades de un equipo.

En nuestro equipo de 8 desarrolladores, en la mayoría de los RP tenemos 1 revisor, que sugiere cambios y finalmente aprueba el RP. Durante la fase de revisión, "cesionario" es la persona que abrió el RP; más adelante, si otro desarrollador recoge PR, se agrega un nuevo "cesionario". Una vez que PR se aprueba y está listo para QA o fusión directa, se agrega un nuevo "cesionario" de QA. De esta manera crece la lista de "cesionarios".

Usamos "cesionario" para designar colectivamente a las siguientes personas:

  1. Autorizar solicitud de extracción
  2. Autor trabajando en sugerencias de cambio de relaciones públicas (generalmente igual que 1)
  3. QA persona involucrada
  4. Persona responsable de fusionar (generalmente igual que 2 o 3)

El uso de "cesionario" ayuda a ubicar el RP en el futuro fácilmente. Uno de mis proyectos tiene> 3000 PR.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

O simplemente author:raya-dumaspara encontrar todos los artículos creados por el autor (problemas, relaciones públicas)

y otras consultas similares para facilitar el proceso de búsqueda. Los "hitos" también son muy útiles para facilitar la búsqueda de relaciones públicas.

Captura de pantalla de Github, cuarto trimestre de 2017

lal
fuente
Muy bien explicado.
Nitin Gaur
Cabe mencionar que puede buscar el autor: my-github-handle para encontrar qué relaciones públicas ha creado una persona
Wisienkas
1

Antes, GitHub solo tenía un campo de cesionario y ningún campo de revisor. No había distinción en ese entonces, por lo que el campo de cesionario se usaba más comúnmente como un campo de revisión.

Pero úselos de cualquier manera que se adapte a su proyecto.

ericcurtin
fuente