Revisiones de seguimiento
Después de la primera revisión de un pull request, HeyDeer parte de lo que ya revisó. Busca problemas nuevos en lo que ha cambiado, vuelve a comprobar sus hallazgos anteriores y no plantea observaciones que la discusión ya cubre.
Revisar solo los cambios nuevos
Activado de forma predeterminada. Configúralo en Configuración de la organización → Reglas de revisión, en Cómo revisar, o en las Reglas de revisión de un repositorio. Se aplica a las revisiones automáticas y a las solicitadas.
Una revisión posterior examina lo que cambiaron los propios commits del pull request desde la última revisión de HeyDeer completada con éxito, y vuelve a comprobar los hallazgos de esa revisión. Los hallazgos que siguen siendo válidos se mantienen sin un comentario nuevo, así que la comprobación y la aprobación siguen cubriendo todo el pull request.
Los commits fusionados desde la rama de destino se excluyen, pero las resoluciones de conflictos se revisan. Las comprobaciones del repositorio añadidas o editadas desde la última revisión se aplican a todo el pull request.
Cuándo revisa HeyDeer todo el pull request
- La primera revisión, o cualquier revisión mientras “Revisar solo los cambios nuevos” esté desactivado.
- Alguien comentó
@heydeer full review. - La rama de destino cambió, o la última revisión fue del mismo commit.
- La última revisión tuvo una advertencia de revisión.
- GitHub ya no tiene el último commit revisado, por ejemplo tras un force push.
El resumen de la comprobación y la fila Revisado de la revisión en HeyDeer indican si una revisión cubrió Cambios nuevos o el PR completo.
Discusión existente
HeyDeer lee la descripción, los comentarios, los resúmenes de revisión y todos los hilos de revisión antes de publicar. No vuelve a publicar un problema que alguien ya planteó, pero lo cuenta como problema hasta que se corrige o se descarta. Una regresión o un problema causado por una corrección cuenta como nuevo.
HeyDeer solo responde en los hilos que inició y solo resuelve esos. Con Marcar como resueltos los hilos de HeyDeer atendidos activado, la opción predeterminada, los resuelve cuando el código actual corrige la observación o demuestra que era errónea, o cuando la observación se descarta. No basta con una respuesta que diga que está corregido.
Descartar un hallazgo
Responde en el hilo del hallazgo y explica por qué, por ejemplo que el comportamiento es intencionado o que se sigue en un issue, o resuelve el hilo. Solo las personas con acceso de escritura o superior pueden descartar; las demás respuestas se leen como contexto.
HeyDeer tiene en cuenta la decisión en la siguiente revisión y la acepta salvo que tenga pruebas concretas de un problema grave que la decisión pasó por alto. Un descarte aceptado no se vuelve a publicar, no cuenta como problema y aparece en Descartadas en la discusión.
Hilos sin resolver
Los hilos de HeyDeer cuya observación sigue abierta o no se pudo verificar aparecen en el resumen de la comprobación, bloquean la aprobación automática y hacen fallar la comprobación cuando Marcar la comprobación como fallida si se detectan problemas está activado.