javascript-tutorial
Proceso de Revisión
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
¿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.
Click en el engranaje y se puede agregar reviewer: Request up to 15 reviewers
@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
help.github.com/en/github/managing-your-work-on-github/assigning-issues-and-pull-requests-to-other-github-users no sirve https://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.