Repository navigation
Proceso de Revisión #286
Description
Para coordinar las review que estamos realizando, creo que necesitamos un procedimiento bien delineado:
-
Solicitar un Reviewer
cualquiera de los miembros: @vplentinax @joaquinelio @EzequielCaste @lizzie136 @ricardov03
Si eres miembro y vas a hacer la revisión, por favor asignarse el PR antes de comenzar. -
El reviewer revisa y corrige errores si existen.
Con el botón "Add your review"
Tienen la opción de Comentar, Aprobar o Solicitar Cambios. -
Solicitar Cambios (Request Changes)
Recomiendo utilizar la sugerencia de cambios para hacer la tarea más eficiente. Es fácil y rápido sugerir un cambio y el autor con un click puede incluir esa sugerencia al PR. -
Aprobación
Una vez que el traductor realiza los cambios solicitados, en reviewer aprueba la traducción. El bot le colocará el tag de "needs +1" que nos ayudará a ver los PR que están aprobados y listo para merge.
Los tags nos deberían servir para ver el estado de los PR:
needs review >> changes requested >> needs +1
Activity
joaquinelio commentedon Jun 29, 2020
¿Asignarse? ¡Oh!, Aprendí algo nuevo, minimizaría la superposición, PERFECTO, dar control a Don BOT y requerir menos coordinación humana, gracias.
¿Qué hay con reviewers invitados?
Hoy por hoy necesitamos más revisión que traducción, es frustrante aportar traducción y que no te respondan en meses.
Si sabe hescrivir vien y detectar errores de concepto de js, ¡100% de aumento! De trabajo.
EzequielCaste commentedon Jun 29, 2020
Click en el engranaje y se puede agregar reviewer: Request up to 15 reviewers
joaquinelio commentedon Jun 29, 2020
@ezzep66
Veo 2 cosas y no sé la diferencia,
reviewers
assignees
o para que sirve "assigne" , crei que era lo mismo y te referias a ese.
en un pr aparezco como reviewer no se si alguien me solicitó o cuando terminé git lo puso solo,
y me pude agregar como assignee "assign yourelf" pero ni idea de para qu´e
el doc no ayuda nada
joaquinelio commentedon Jun 29, 2020
help.github.com/en/github/managing-your-work-on-github/assigning-issues-and-pull-requests-to-other-github-users
no sirve
https://github.com/proxy/developer.github.com/v3/issues/assignees/
no sirve
google "whats the point in github assignee"
stackoverflow:
actually meant to be used with issues rather than pull request so that when we receive a issue we can assign someone to fix it
OK!!!! Ahora entiendo que efectivamente te referías a reviewer.