Code review

Regla vigente

Mínimo 1 aprobación antes de hacer merge.

Fuente

Convención general de la organización, definida en Industrias-CTS/.github.

Pendiente de definir

Pendiente

• ¿Quién debe aprobar? ¿Cualquiera del equipo, o el owner de la aplicación? Ver Mapa de ownership.
• ¿Hay tipos de cambio que exijan más de una aprobación (esquema de datos, seguridad, infraestructura)?
• ¿Qué tiempo de respuesta se espera para revisar un PR?
• ¿Qué se revisa? Un criterio compartido evita que cada review mire cosas distintas.
• ¿Qué pasa con un hotfix urgente? ¿Se puede mergear y revisar después?

Por qué importa cerrar esto

Con tres aplicaciones sin owner asignado, hoy no está claro quién puede aprobar cambios sobre ellas. Resolver el ownership es requisito para que esta regla funcione.