Reviewers role

[lizzie136]

cc./ @tscandalitta @caballerof @maurodibert @victorze

Hola!

este issue es para ordenarnos un poco en cuanto a la forma en que se revisan los PR.
Como reviewers, deberán poder dejar comentarios en los pull request para que las personas que los hayan mandado los corrijan.

Puede tardar un poco más la traducción, pero por otro lado este proyecto es open source y esta como good first project to contribute para muchas personas. Esto implica que tenemos que guíar a las personas en el proceso que sigue un Pull Request.

Reforzar 💪

• Comentarios en los pull request, indicando que pueden mejorar, que deben cambiar antes de dar merge a sus PR.
• Ningún PR y commit de traducciónes debería agregarse a master hasta no haber sido revisado y corregido.
• Estructura de los commits. En este momento tenemos decenas de commits que son como: fix, update, corregir, etc etc sin mucho contexto.
  Por ello les pediría que sus commits y los de las personas que están contribuyendo tengan la siguiente estructura:

   (unidad:sección:pagina markdown) - detalle de lo que hace el commit
  

  Por ejemplo:

  (js:logical-operators:check-login/solution.md) - actualiza formato de código
  

  Si necesitan agregar más detalle:

   (js:logical-operators) - primera traducción
  
      - primera traducción de article.md
      - se tradujo comentarios y texto de ejercicio checklogin
  

Actualizaré en CONTRIBUTING.md para tener en cuenta este detalle sobre los commits que es importante para ordenarnos.

Glosario

Algunas palabras en las que ya nos habíamos puesto de acuerdo pero como para recordarlas:

• run => ejecutar
• script => script

Si recuerdan algunas más o quieren conversar de alguna conversemoslo en este issue 😄

Compartir

Sería genial poder incluir a muchas mas personas en la traducción y si uds conocen a alguien que sea un buen reviewer también presentelo. Este es un proyecto de todos para todos 🙌

[tscandalitta]

Perfecto, tengo un par de dudas.
Podemos corregir los PR también o solamente hacer comentarios para que los corrijan los autores?
Y con respecto a los merge, basta con que solo uno de nosotros revise una traducción y le parezca correcta o deberíamos revisarla todos y coincidir en que ya esta lista para agregar a master?

[lizzie136]

@tscandalitta la idea es que lo corrijan los/las autores/as, para eso le dejamos los comentarios en el review. Puedes dejar el PR en request for changes hasta que corrija lo que sea necesario. Creo que si vamos a editar directamente nosotros, es luego de darle la oportunidad a la persona que mando el Pull Request de corregirlo. De lo contrario, personalmente, da la sensación de pasivo agresivo. ¿Qué opinan?

[maurodibert]

Elizabeth, coincido plenamente con vos! Me parece más enriquecedor para todos el sugerir mejoras y la persona pueda avanzar en su proceso mientras avanza la traducción. El mié., 22 de may. de 2019 a la(s) 11:54, Elizabeth Portilla ( notifications@github.com) escribió:

…

@tscandalitta <https://github.com/tscandalitta> la idea es que lo corrijan los/las autores/as, para eso le dejamos los comentarios en el review. Puedes dejar el PR en request for changes hasta que corrija lo que sea necesario. Creo que si vamos a editar directamente nosotros, es luego de darle la oportunidad a la persona que mando el Pull Request de corregirlo. De lo contrario, personalmente, da la sensación de pasivo agresivo. ¿Qué opinan? — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#48?email_source=notifications&email_token=AGJW6FWOJONU5IIA7VJX6YTPWVNDJA5CNFSM4HOU7QF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV7KGQY#issuecomment-494838595>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGJW6FSGX6GC4B2RXL4SWS3PWVNDJANCNFSM4HOU7QFQ> .

-- Mau.-

[lizzie136]

Esta conversación ya tiene un par de años abierta. Voy a darla por cerrada ya que las personas que están contribuyendo en este momento tienen su propio flujo de trabajo.