PR fáciles de revisar
Los hábitos que ayudan a tus compañeros a revisar un pull request también ayudan a HeyDeer.
Mantén cada pull request centrado
Un pull request que hace una sola cosa recibe hallazgos sobre esa cosa. Separa las refactorizaciones, los cambios de nombre y los movimientos de archivos de los cambios de comportamiento, porque entierran las líneas que realmente cambiaron.
Describe la intención y vincula el issue
HeyDeer lee el título y la descripción. Indica para qué es el cambio, la parte de la que estás menos seguro y lo que dejaste fuera a propósito. Cuando el autor tiene acceso de escritura, una concesión explícita como “Los reintentos quedan fuera del alcance; se siguen en #412” cuenta como una decisión, igual que descartar un hallazgo.
Vincula también el issue. Con “Leer issues vinculados” activado, como está de forma predeterminada, HeyDeer puede leerlo para conocer los requisitos que debe cumplir el cambio. Consulta Qué lee HeyDeer.
Marca los archivos generados
Marca el código generado con linguist-generated en .gitattributes y HeyDeer lo excluirá. HeyDeer lee el atributo desde la rama base, así que fusiona antes ese cambio en su propio pull request.
Abre un borrador mientras iteras
Las revisiones automáticas omiten los borradores de forma predeterminada y tienen en cuenta el pull request cuando lo marcas como listo para revisión. Para obtener comentarios tempranos, comenta @heydeer review en el borrador. Consulta Filtros de revisión.
Sube las correcciones como commits nuevos
Las revisiones de seguimiento examinan lo que ha cambiado desde la última revisión, lo que funciona mejor cuando las correcciones llegan como commits nuevos. Fusionar la rama de destino o hacer rebase sobre ella no supone ningún problema. Haz push de las correcciones relacionadas a la vez para usar menos revisiones automáticas. Consulta Revisiones de seguimiento.
Responde a los hallazgos que descartes
Responde en el hilo con el motivo, no solo “no se corregirá”, o resuelve el hilo. Una decisión de alguien con acceso de escritura le indica a HeyDeer que no vuelva a plantear el problema. Consulta Reducir el ruido.