Buenas prácticas
Reducir el ruido
Si HeyDeer publica más de lo que tu equipo quiere leer, busca el tipo de ruido más abajo y aplica su solución. Las primeras filas son las que más cambian con menos esfuerzo.
Soluciones
| Ruido | Solución | Detalles |
|---|---|---|
| Demasiados hallazgos que no atenderías | Cambia el Alcance de los comentarios a Conciso. Para comparar alcances antes, abre una revisión reciente en HeyDeer y usa Vista previa del alcance de los comentarios. | Alcance de los comentarios |
| Comentarios sobre código generado | Marca los archivos como linguist-generated en .gitattributes en la rama base. | Qué lee HeyDeer |
| Revisiones de pull requests de bots o desechables | Añade filtros de revisión automática, como Cuenta del autor → No es ninguno de → Bot, o Etiquetas del PR → No es ninguno de → skip-review. | Filtros de revisión |
| Un hallazgo que decidiste no corregir | Responde con el motivo o resuelve el hilo. Una decisión de alguien con acceso de escritura hace que HeyDeer no vuelva a plantearlo. | Revisiones de seguimiento |
| Un tipo de comentario que nadie atiende | En las instrucciones de revisión o en AGENTS.md, indica sobre qué no comentar, de forma concreta y con un motivo. Evita reglas amplias como “No comentes sobre el rendimiento”. | Cómo escribir pautas de revisión |
| Una comprobación que se activa en archivos que no le corresponden | Acota sus patrones files y excluye los tests o los fixtures. | Cómo escribir comprobaciones eficaces |
| Revisiones de seguimiento en cada push | Mantén “Revisar solo los cambios nuevos” activado y establece un “Máximo de revisiones automáticas por PR” o usa “Una vez por PR”. | Revisiones automáticas |
| Una conversación larga en el pull request | Mantén “Marcar como resueltos los hilos de HeyDeer atendidos” activado, o desactiva “Publicar comentarios de revisión en el PR” para que la comprobación HeyDeer Review contenga la revisión. | Publicación de resultados |
Ejemplos
Archivos generados en .gitattributes:
src/generated/** linguist-generated
*.pb.go linguist-generated=trueReglas concretas sobre qué dejar tal cual:
## Leave alone
- Files under scripts/one-off/. They run once by hand and are deleted.
- Missing input validation in internal/admin-cli/. It only runs
locally with trusted input.