-
Notifications
You must be signed in to change notification settings - Fork 222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proceso de Revisión #286
Comments
¿Asignarse? ¡Oh!, Aprendí algo nuevo, minimizaría la superposición, PERFECTO, dar control a Don BOT y requerir menos coordinación humana, gracias. |
@ezzep66 en un pr aparezco como reviewer no se si alguien me solicitó o cuando terminé git lo puso solo, |
help.github.com/en/github/managing-your-work-on-github/assigning-issues-and-pull-requests-to-other-github-users google "whats the point in github assignee" |
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
The text was updated successfully, but these errors were encountered: