Categorías
Sobre “una prueba que no puede fallar”
Nunca se reportan pruebas faltantes. Sí se reporta, comologic, una prueba en el diff que seguiría pasando aunque se rompiera el comportamiento que nombra: por ejemplo una aserción sobre una colección que la prueba nunca llena, o una aserción cuyos dos lados son la misma expresión. El comentario dice qué entrada hace que pase mientras el código está roto.
Gravedades
Una gravedad eshigh, medium o low (alta, media o baja): el juicio del propio modelo sobre qué tan grave es el defecto. Todo hallazgo sigue siendo un defecto; low nunca es una observación de estilo.
La gravedad no decide qué se publica. Se publican todas las gravedades. Los hallazgos se ordenan por gravedad y luego por la confianza del modelo, y el tope de comentarios de profile conserva la parte superior de esa lista. Un hallazgo low se publica solo cuando nada peor necesita el espacio.
Esto es deliberado. En un pull request, la misma consulta N+1 en la misma línea del mismo commit salió medium en tres ejecuciones y low en otras tres. Una etiqueta tan inestable no puede ser lo que decida si un defecto real te llega.
Confianza
El modelo también da a cada hallazgo una confianza entre 0 y 1. Los hallazgos por debajo de 0,7 no se publican. La confianza de cada comentario, publicado o descartado, se muestra en la página de la revisión en la consola.Lo que nunca es un comentario
Estilo, nombres, formato, orden de imports, una prueba faltante, documentación faltante, una anotación de tipo faltante, sugerencias que empiezan con “considera” y cualquier cosa sobre código que no se le mostró al modelo. Se rechazan por diseño y no se pueden activar en.codewarden.yml.
Reaccionar a un comentario
Reacciona con 👍 o 👎 a un comentario para decir si ayudó. La página de mediciones muestra cuántas reacciones 👎 recibieron los comentarios contados. Para discutir con un comentario, responde con@codewardenapp: consulta Hablar con CodeWarden.