Flake8 F824 for global variable scoping
I maintainer di solito rispondono entro 1 giorno
@cpswan ci sta già lavorando.
Dal 25/4/2025.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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
- Lingua principale
- Python
- Stelle
- 0
- Fork
- 4
- Merge medio
- 6h 25m
- PR unite (30g)
- 11
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di atsign-foundation/at_python
-
Mature Python SDKForse di nuovo libera @Xlin123 l’ha presa 344 giorni fa e non c’è nessuna pull request aperta. Aperta
atsign-foundation/at_python#413 · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
atsign-foundation/at_python#87 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Add notification managementForse di nuovo libera @shahumang19 l’ha presa 1184 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
atsign-foundation/at_python#8 · 1 commento · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di atsign-foundation/at_python
Issue simili
-
Claiming namespace `apoint`Apertanamespace operations
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
EclipseFdn/open-vsx.org#13573 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
collective/icalendar#1854 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
rancher/rancher-ai-agent#412 ·
I maintainer di solito rispondono entro 6 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
TUDelftGeodesy/DePSI#134 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
HenriquesLab/rxiv-maker#335 ·