Issues y PRs
- Cada PR debe tener asociada la issue correspondiente para marcarla como "Done".
- Haz PRs y commits pequeños, facilitan la revisión.
- Comenta cambios y responde a solicitudes de revisión en GitHub.
- Para discusiones largas, crea un hilo en Discord y avisa a quien corresponda.
Revisiones
- Toda PR necesita revisión antes de mergear.
- Si una review no es aprobada, explica claramente la razón.
- No es personal: una PR rechazada no refleja tu habilidad.
- Revisiones superficiales están prohibidas, revisa de verdad.
Commits y ramas
- Usa Conventional Commits:
feat:, fix:, etc.
- Mensajes claros y descriptivos, evita genéricos como “cambios varios”.
- Ramas cortas y descriptivas (
feature/login-form), no nombres genéricos.
PR y descripción
- Título de PR claro.
- Descripción con contexto, pasos para probar y screenshots si aplica.
Status checks y CI
- Todos los checks (tests, lint, CI/CD) deben pasar antes de mergear.
- Avisar en Discord si algo falla y no mergear hasta solucionarlo.
Cultura y comunicación
- Mantén respeto y buen rollo en PRs y Discord.
- Evita discusiones largas en GitHub, muévelas a Discord si es necesario como hilo en la notificación de la PR.
- Recuerda: tu código no eres tú, una PR rechazada no te hace mal programador ❤️.