Flake8 F824 for global variable scoping
Les mainteneurs répondent en général sous 1 jour
@cpswan y travaille déjà.
Depuis le 25/4/2025.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
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
- Langage dominant
- Python
- Étoiles
- 0
- Forks
- 4
- Merge moyen
- 3 h 52 min
- PR mergées (30 j)
- 17
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de atsign-foundation/at_python
-
Mature Python SDKPeut-être à nouveau libre @Xlin123 l’a pris il y a 378 jours, et aucune pull request n’est ouverte. Ouverte
atsign-foundation/at_python#413 · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
atsign-foundation/at_python#87 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Add notification managementPeut-être à nouveau libre @shahumang19 l’a pris il y a 1218 jours, et aucune pull request n’est ouverte. Ouverteenhancement
atsign-foundation/at_python#8 · 1 commentaire · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de atsign-foundation/at_python
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Nixtla/mlforecast#823 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
microsoft/ML-For-Beginners#1061 ·
-
bot-found documentation priority: P3
Difficulté 2/5 Moins d'une heure Accessibilité débutants 82/100
madenvel/KalinkaPlayer#503 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
EleutherAI/lm-evaluation-harness#4508 ·
Les mainteneurs répondent en général sous 1 jour