Skip to content
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

Open
EzequielCaste opened this issue Jun 29, 2020 · 4 comments
Open

Proceso de Revisión #286

EzequielCaste opened this issue Jun 29, 2020 · 4 comments

Comments

@EzequielCaste
Copy link
Contributor

EzequielCaste commented Jun 29, 2020

Para coordinar las review que estamos realizando, creo que necesitamos un procedimiento bien delineado:

  1. 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.

  2. 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.

  3. 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.

  4. 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

@joaquinelio
Copy link
Member

¿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
Copy link
Contributor Author

image

Click en el engranaje y se puede agregar reviewer: Request up to 15 reviewers

@joaquinelio
Copy link
Member

@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
Copy link
Member

joaquinelio commented 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://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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants