Revisión automática de código - JuanHerreraP/Proyecto-MISO-Solventa-Grupo-11 GitHub Wiki

Revisión automática de código

El proyecto cuenta con revisión automática de código mediante el pipeline de Integración Continua en GitHub Actions.

Para el backend, el workflow ejecuta:

• ruff check .: analiza reglas de calidad estática, como imports desordenados, variables sin uso, líneas excesivamente largas y patrones de manejo de excepciones.
• ruff format --check .: valida que el formato del código sea consistente.

Adicionalmente, cuando existen los proyectos correspondientes, el pipeline ejecuta npm run lint para la aplicación web y ./gradlew lint para Android.

Estas validaciones se ejecutan en cada Pull Request y en cada push a main, permitiendo identificar problemas de calidad antes de integrar cambios al repositorio. La evidencia se consulta en los resultados del workflow de CI de GitHub Actions.

Ver configuración del pipeline de CI

Ejemplo de hallazgos detectados

En una ejecución del pipeline, Ruff identificó dos tipos de hallazgos antes de integrar cambios:

• I001: imports sin ordenar. Por ejemplo, el archivo tests/perfilamiento/conftest.py tenía los imports internos de app en un orden distinto al esperado.
• B904: una excepción HTTP generada dentro de un bloque except debía indicar explícitamente que no conserva la causa original usando from None.

except ValueError:
    raise HTTPException(
        status_code=status.HTTP_403_FORBIDDEN,
        detail="Rol no reconocido.",
    ) from None