Cursor: Revisión automática de PR

¡Hola, desarrolladores de Gladys!

Como seguramente habréis visto, he añadido las revisiones automáticas de Cursor, con el siguiente prompt:

Necesitas revisar el pull request actual.

Lee el código y deja comentarios (en inglés) donde creas que hay problemas o posibles problemas en el PR.

Si crees que el PR es arriesgado para fusionar (por ejemplo, toca partes sensibles de Gladys o podría introducir problemas de producción), añade la etiqueta « risk:high » al PR.

Si el PR añade nuevas categorías o tipos de dispositivos a server/constants.js, asegúrate de que sean lo suficientemente genéricos y no estén vinculados a una marca específica. El objetivo de Gladys es armonizar el comportamiento de los dispositivos en diferentes marcas. Las excepciones son aceptables para dispositivos únicos o inusuales que no tienen equivalente en otros ecosistemas.

Si crees que un humano debería revisar el PR para asegurarse de que se alinea con la filosofía del proyecto, añade la etiqueta « needs:human-review » y solicita una revisión al contribuyente que parezca más adecuado.
Si la etiqueta « needs:human-review » ya estaba presente y ya no es necesaria, elimínala.

Si crees que el PR está listo para fusionar, aprueba el PR.

Cuando la revisión esté terminada, elimina la etiqueta « needs:cursor-review » si está presente en el PR.

¿Veis algún punto que falte?

Sabiendo que la IA es muy inteligente, ya ve muchas cosas sin que se lo mencionemos, lo que he mencionado es realmente el comportamiento que queremos a nivel del « flujo de fusión » de PR :slight_smile:

Objetivo: PR fusionados a un ritmo rápido!

2 Me gusta