Reglas
Bienvenid@ al PR Review Club en ESP, una iniciativa de Librería de Satoshi para ayudar a devs a empezar revisando PRs de Core.
¿Cómo proponer un PR?
- Cualquiera puede proponer un PR para revisar. La persona que quiera sugerir un PR deberá mandarme un DM o mencionarme en el chat general diciendo que quiere sugerir un PR.
- La persona que quiera sugerir un PR también será la encargada de preparar las preguntas previas y de liderar la sesión.
- No cualquier PR será aceptado, debe ser un PR suficientemente interesante para que el resto de participantes aprenda, por ello Pol Espinasa va a revisar dicho PR de antemano.
¿Qué es un PR review club?
- En un PR review club un grupo de personas se juntan para revisar, discutir y hablar sobre un PR en concreto.
- Dichos clubs están guiados por una persona que habrá estudiado el PR previamente y habrá propuesto una serie de preguntas que sirven a los otros participantes como guion.
- Los otros participantes deberían hacer un trabajo previo de revisión del PR y preparar las respuestas a las preguntas propuestas.
- Durante la sesión de revisión se discutirán las respuestas a las preguntas y otros temas que puedan aparecer.
- La duración de las sesiones será de 1 hora como mucho.
- Las sesiones se llevarán a cabo por chat escrito.
Qué no hacer antes, durante y después de un PR review club
- Las faltas de respeto no se tolerarán y llevarán al ban permanente del canal.
- Intentar no monopolizar las conversaciones, dejar espacio a todo el mundo para comentar.
- No bombardear los PRs con cACK, ACK, NACK, approach ACK, etc. GitHub es el espacio de trabajo de los contribuidores de Bitcoin Core, e ir a escribir reviews pobres solo genera ruido y molesta. Exclusivamente se debería comentar en los PRs las reviews que se hayan considerado interesantes. No hay que ir todos los participantes del PR review club a dejar una review ACK solamente porque se haya participado en el club.
¿Te apetece participar en la próxima sesión? Únete ya a nuestro Discord.