Revisiones de seguimiento
Después de la primera revisión de un pull request, HeyDeer revisa lo que ha cambiado y vuelve a comprobar los problemas anteriores. Descubre cómo funciona y cómo descartar un hallazgo.
Revisar solo los cambios nuevos
Revisar solo los cambios nuevos está activado de forma predeterminada. Lo encontrarás en Configuración del espacio de trabajo → Reglas de revisión → 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 solo busca hallazgos nuevos en lo que cambiaron los propios commits del pull request desde la última revisión de HeyDeer completada correctamente. También vuelve a comprobar los hallazgos de esa revisión, y los que siguen siendo válidos cuentan para la comprobación y la aprobación.
Los commits incorporados desde la rama base mediante merge o rebase 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.
El recorrido de una revisión posterior sigue cubriendo todo el PR y marca los grupos cuyos archivos cambiaron desde la última revisión.
Problemas anteriores
Un problema anterior es un hallazgo que HeyDeer planteó en su propia conversación de revisión en línea en una revisión anterior. Cada revisión posterior vuelve a comprobar los que siguen abiertos con el código actual.
Los problemas que siguen siendo válidos aparecen en Problemas anteriores que atender. Las conversaciones anteriores que HeyDeer no puede vincular a un hallazgo aparecen en Problemas anteriores por confirmar; resuélvelas cuando ya no apliquen.
Los problemas anteriores abiertos o sin confirmar aparecen en el resumen de la comprobación y bloquean la aprobación automática. Con Marcar la comprobación como fallida si se detectan problemas activado, también hacen fallar la comprobación.
HeyDeer solo resuelve las conversaciones que inició. Con Marcar como resueltas las conversaciones de HeyDeer atendidas activado, la opción predeterminada, las resuelve cuando el código actual corrige el problema o demuestra que era erróneo. No basta con una respuesta que diga que está corregido.
Los cambios fusionados desde un PR de corrección se vuelven a comprobar de la misma forma en la siguiente revisión del PR original.
Descartar un hallazgo
Responde en la conversación del hallazgo y explica el motivo, por ejemplo que el comportamiento es intencionado o que se sigue en un issue, o resuelve la conversación. Cuando alguien con acceso de escritura o superior lo descarta, HeyDeer deja de revisar el problema y no lo vuelve a plantear, aunque el código siga teniéndolo. Las demás respuestas se leen como contexto.
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 base cambió desde la última revisión.
- Alguien solicitó otra revisión de un commit que HeyDeer ya había revisado, por ejemplo después de añadir contexto.
- La última revisión tenía notas de la revisión sobre hallazgos omitidos, demasiadas comprobaciones aplicables o una evaluación incompleta, o pidió una aclaración.
- GitHub ya no tiene el último commit revisado.
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 del PR como contexto. Los comentarios, el cuerpo de las revisiones y las conversaciones de revisión de otras personas nunca contienen un problema anterior ni impiden que HeyDeer publique un hallazgo.