Saltar a contenido

Checklist de Pull Request — GekkoCheckin

Antes de abrir el PR

  • Código compila: cargo build (backend), npm run build o ionic build (frontend).
  • Tests existentes pasan: cargo test, ng test (si hay specs).
  • Sin unwrap()/expect() en rutas de request (backend); suscripciones con takeUntil(destroy$) donde aplique (frontend).

Contenido del PR

  • Frontend: cambios de estilo usan variables Ionic/theme; no se añaden hex/rgb en SCSS de componentes sin justificar (ver styles.md).
  • Frontend: componentes/páginas nuevas tienen spec mínimo (should create) y, si son críticos, data-testid donde aplique (ver testing.md).
  • Backend: nuevos handlers o lógica relevante usan AppError y al menos un log de tracing en operaciones importantes (ver error-handling.md).
  • Backend: no se importa infrastructure::persistence directamente desde handlers; se usa crate::repository (ver hexagonal.md).
  • Actividad: si se añade una acción auditable, se llama a activity_log::append_activity y se documenta el evento (ver activity-log.md).

Descripción

  • Título corto y descriptivo.
  • Qué problema resuelve o qué feature añade.
  • Si cambia API (rutas, payloads), indicarlo y actualizar OpenAPI/docs si existe.

Revisor

  • Revisar que el checklist anterior esté cubierto.
  • Aprobar solo si los mínimos de definition-of-done.md se cumplen para el alcance del PR.