javascript-tutorial / javascript-tutorial/es.javascript.info
Proceso de Revisión
- Dominant language
- HTML
- Stars
- 625
- Forks
- 226
- PR merge metrics
- No merged PRs in 30d
Description
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**
Contributor guide
Research direction
The issue names no file, test, or entry point. Start by comparing the proposed review steps with the repository's current pull-request labels and bot behavior; done would be an agreed, current procedure and a clear decision about where it should be documented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github
- Domain
- developer-experience, documentation
- Issue type
- Documentation
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100