Flake8 F824 for global variable scoping
Los mantenedores suelen responder en 1 día
@cpswan ya está trabajando en esto.
Desde el 25/4/2025.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Describe the bug
#355 introduced Flake8 7.2.0, which added a new variable scoping check. at_client currently fails that test throwing an F824 error:
./at_client/atclient.py:348:13: F824 `global should_be_running_lock` is unused: name is never assigned in scope
global should_be_running_lock
^
./at_client/atclient.py:371:13: F824 `global should_be_running_lock` is unused: name is never assigned in scope
global should_be_running_lock
^
./at_client/connections/atmonitorconnection.py:37:9: F824 `global should_be_running_lock` is unused: name is never assigned in scope
global should_be_running_lock
^
./examples/repl.py:93:17: F824 `global shared_queue` is unused: name is never assigned in scope
global shared_queue
^
4 F824 `global should_be_running_lock` is unused: name is never assigned in scope
4
https://github.com/atsign-foundation/at_python/pull/355/commits/d7f839491e35269f2ec0121bb5200117e6d44157 put in place a workaround, and this ticket is to track its eventual removal.
Steps to reproduce
Run flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics --exclude .venv,.git without --ignore=F824
Expected behavior
Lints pass clean
- Lenguaje dominante
- Python
- Estrellas
- 0
- Forks
- 4
- Merge medio
- 6 h 25 min
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 atsign-foundation/at_python
-
Mature Python SDKQuizá libre de nuevo @Xlin123 la tomó hace 345 días y no hay ningún pull request abierto. Abierto
atsign-foundation/at_python#413 · 2 asignados ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
atsign-foundation/at_python#87 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Add notification managementQuizá libre de nuevo @shahumang19 la tomó hace 1185 días y no hay ningún pull request abierto. Abiertoenhancement
atsign-foundation/at_python#8 · 1 comentario · 2 asignados ·
Los mantenedores suelen responder en 1 día
Todos los issues de atsign-foundation/at_python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
kornia/kornia#5263 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Metadata correction for W16-5400Abiertoapproved correction metadata
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
acl-org/acl-anthology#10133 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
BasedHardware/omi#20084 ·
Los mantenedores suelen responder en 1 día
-
bug needs-acceptance wg/evaluation-quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
vllm-project/semantic-router#4424 ·
Los mantenedores suelen responder en 1 día