Solicitudes de extracción

Cómo hacer una buena solicitud de extracción

Los Pull Requests (PR) son la forma en que envías tus cambios propuestos a alguna parte del código base de Exercism. Si estás leyendo esto y estás armando un PR, ¡gracias! Esta guía busca ayudarte con la parte no técnica de los PR, para que tengas una gran experiencia y al mismo tiempo ayudes a nuestro equipo de mantenedores.

Qué esperar

El proceso para los PR es el siguiente:

  • Envías un PR
  • Uno o más mantenedores revisan el PR y dejan comentarios con sugerencias. En algunos casos, pueden discutir el PR en su totalidad para entender si es lo adecuado para agregar.
  • Tu PR se fusiona y recibes Reputation en el sitio web. O tu PR se cierra.

¿Cómo logro que fusionen mi PR?

Hay ciertas cosas que puedes hacer de antemano para que sea más probable que fusionen tu PR con cambios mínimos.

Lo más importante es asegurarte, antes de enviar el PR, de que los mantenedores recibirán con agrado los cambios que haces. La forma más fácil de lograrlo es abrir primero un Issue en GitHub, explicar qué piensas hacer y preguntar si los mantenedores están abiertos a esos cambios.

Note

Si encuentras un Task en el sitio web de Exercism, no hace falta abrir un Issue nuevo. Puedes comentar en el Issue existente que vas a trabajar en él y luego continuar.

Lo segundo es asegurarte de dejar una descripción clara en el PR cuando lo abres. Los PR sin descripción son increíblemente frustrantes para los mantenedores, porque no tienen contexto al leer el código. Cuéntanos por qué hiciste los cambios, enlaza el Issue del que trata el PR, explica cualquier disyuntiva que hayas tenido que asumir, etc.

Por último, asegúrate de que tu PR solo contenga cambios relacionados con una única mejora. Si tienes varias mejoras, repártelas en varios PR. Así los mantenedores pueden revisar más fácil y más rápido, y tu vida también será más sencilla.

¿Cuánto debo esperar?

Normalmente los mantenedores revisan tu PR en unos pocos días. Por favor, no estés subiendo el PR con comentarios adicionales durante la primera semana. Nuestros mantenedores son voluntarios y personas muy ocupadas, y lo revisarán tan pronto como puedan. Si no recibiste respuesta en 7 días, menciona a @exercism/reviewers en un comentario nuevo en el PR, y eso alertará a un equipo más amplio.

Gracias de nuevo por contribuir a Exercism. ¡Esperamos que sea una experiencia divertida y valiosa!