Repository navigation
Proceso de Revisión #286
Description
Activity
¿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 nadahelp.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 sirvegoogle "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.
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