Una verificación es una verificación individual. Una ejecución es la ejecución de las verificaciones en una política aplicada a un cambio. Un hallazgo es algo que una verificación reportó.
Los hallazgos de una tarea se encuentran en su sección de Control de Calidad , y cada ejecución se gestiona desde la Configuración → Actividad.
Leer una ejecución
Una ejecución muestra cada verificación, su resultado y su salida. Cuando una tarea no supera la verificación, la ejecución te indicará el motivo.
Léela de arriba abajo. La primera verificación que falle suele ser la verdadera; los fallos posteriores son a menudo consecuencias. Un error de compilación seguido de doce fallos de prueba representan un único problema, no trece.
Hallazgos
Los hallazgos provienen de agentes revisores, escáneres y puertas de acceso consultivas. Cada uno indica lo que se encontró, dónde y qué tan grave es. Un hallazgo en una tarea permanece vinculado a esa tarea. Un fallo de prueba que no pertenece a ninguna tarea espera en la Fábrica bajo Revisión, donde puedes corregirlo o aceptarlo con una justificación. Una comprobación que no puede ejecutarse en un proyecto, como pruebas unitarias sin un entorno de pruebas, lo indicará en su fila en Control de Calidad.
Su ciclo de vida:
- Generado por una verificación o un revisor.
- Enrutado — los hallazgos solucionables se devuelven al agente desarrollador como trabajo adicional; el resto se vinculan al pull request.
- Resuelto — corregido, aceptado o suprimido.
Detectados
- Una puerta de aviso
El agente revisor- Un escáner conectado
Enrutados
Solucionables
Se devuelve al desarrollador como trabajo pendiente
Todo lo demás
Se adjunta al pull request
Resueltos
Corregidos
El cambio lo resuelve
Aceptados
Es cierto, pero no merece acción aquí. Indica el motivo.
Suprimidos
Es un falso positivo o una regla que no aplica. Indica el motivo.
Clasificación
La pregunta para cada hallazgo no es "¿es cierto?" sino "¿cambia esto lo que sucederá después?"
- Corregir ahora — corrección, seguridad o cualquier cosa que de otro modo haría fracasar la revisión.
- Aceptar — es cierto pero no merece acción en este momento. Registra el motivo.
- Suprimir — un falso positivo o una regla que no aplica a este código.
Suprime deliberadamente y con un motivo. Una supresión sin explicación es indistinguible de silenciar una verdad incómoda, y seis meses después nadie podrá saber cuál fue.
Cuando hay demasiados hallazgos
El volumen es una señal sobre la configuración, no sobre la calidad del código.
Una verificación que se activa en la mayoría de los cambios no está encontrando problemas; está describiendo tu base de código. O bien la regla no coincide con tus convenciones, o describe una deuda que has decidido aceptar. En ambos casos, la verificación tiene una forma incorrecta.
Hallazgos y puertas de control
Un hallazgo procedente de una puerta de control consultiva es información. El mismo hallazgo procedente de una puerta de control bloqueante detiene la tarea. El hallazgo es idéntico; la política determina su coste.
Esa es la palanca a utilizar cuando una verificación es correcta pero demasiado agresiva: cambia su gravedad en lugar de eliminarla. Consulta Políticas y puertas de control de calidad.