Ir al contenido
Logo de SyncroSyncro Docs

Pull Requests

El PR no es un trámite, es el control de calidad y la principal herramienta de aprendizaje del equipo.

Reglas

Sección titulada «Reglas»
  • Revisión obligatoria — Ningún PR puede mergearse sin al menos 1 aprobación
  • Descripción clara — Explica qué problema resuelve y cómo probarlo
  • Cultura de feedback — Sé constructivo. Comenta oportunidades de mejora
  • Destino correcto — Siempre apunta a dev, nunca a main

CI

Sección titulada «CI»

Desde la v0.5.0, GitHub Actions ejecuta automáticamente en cada PR (y push a dev/main):

  • Lint — pnpm lint (ESLint con --max-warnings 0)
  • Typecheck — pnpm check-types

Ambos jobs son bloqueantes: un PR no se puede mergear con lint o typecheck en rojo. El workflow vive en .github/workflows/ci.yml (usa pnpm 9 y Node 22).

Regla de oro: CI verde obligatorio antes de mergear. Si tu PR rompe dev, lo arreglas tú.