`make static-check` fails on master: Makefile uses --max-complexity=10, CI uses 11
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 82/100
Línea de trabajo
Lee .github/workflows/run-tests.yaml:53, Makefile:42 y scripts/README.md:358 para comparar los límites de complejidad que utilizan CI y la comprobación local. Actualiza los dos comandos locales documentados para que coincidan con el límite de CI y, después, ejecuta make static-check; la tarea se considera completada si termina correctamente, como en CI.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
make static-check fails on a clean master checkout. CI's flake8 step was moved to --max-complexity=11 in 022bdeb8 (.github/workflows/run-tests.yaml:53), but Makefile:42 and the Windows command in scripts/README.md:358 still use 10. Two functions are now at 11, so CI passes while the local check fails. scripts/README.md also suggests running make static-check from a pre-commit hook, and that hook currently blocks every commit.
To Reproduce
- Check out master (94d0d878)
- Run
make static-check - See:
./scripts/convert.py:999:1: C901 'get_template_for_edition' is too complex (11)
./scripts/pdf_generation/merge_pdfs.py:327:1: C901 'main' is too complex (11)
make: *** [static-check] Error 1
Expected behavior
make static-check passes whenever CI's lint step passes.
Desktop
- OS: macOS, Docker 29.9.2
Additional context
Setting 11 in Makefile:42 and scripts/README.md:358 fixes it; make static-check then exits 0. If you'd rather keep 10, I can refactor the two functions instead. Could I be assigned?
- Lenguaje dominante
- Python
- Estrellas
- 158
- Forks
- 110
- Merge medio
- 1 d 11 h
- PR fusionados (30 d)
- 115
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OWASP/cornucopia
-
Upgrade @vitest/mocker from 4.1.10 to 5.0Quizá libre de nuevo @Mahaboobunnisa123 la tomó hace 43 días y no hay ningún pull request abierto. Abiertohelp wanted javascript
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
OWASP/cornucopia#3538 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
OWASP/cornucopia#3784 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
OWASP/cornucopia#3783 ·
Los mantenedores suelen responder en 1 día
-
Add the Hindi translation for the Companion EditionPosiblemente ocupada @V-2007-P la tomó hace 2 días. Abierto
OWASP/cornucopia#3770 · 6 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
bug(copi): next_round and toggle_continue_vote can be triggered on unstarted or finished gamesPosiblemente ocupada @Abhrant-Singh la tomó hace 3 días. Abierto
OWASP/cornucopia#3762 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de OWASP/cornucopia
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
AI bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
AUTO-MAS-Project/AUTO-MAS#1960 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
btclib-org/btclib-wallet#872 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: Scheduled returns scan all accepted minion keys causing extreme loadPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
saltstack/salt#70789 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 87/100
btclib-org/btclib#3093 ·
Los mantenedores suelen responder en 1 día