false-positive `name-defined` regression with dataclasses and shadowing
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 20.6k
- Forks
- 3.3k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug Report
This file:
from dataclasses import dataclass
import collections
@dataclass
class Shadow:
collections: collections.deque
other: collections.defaultdict
print(Shadow(collections.deque(), collections.defaultdict()))
is perhaps a bit unorthodox, but runs fine, because the collections: ... doesn't actually shadow the module when defining other.
Actual Behavior
mypy v0.950 was happy with it, but mypy v0.960 claims that:
test.py:9: error: Name "collections.defaultdict" is not defined
Introducing commit
Bisected to 03901ef657daf6f477e6fe933c42ac270cf5034b ("Running dataclass transform in a later pass to fix crashes (#12762)", @JukkaL)
Your Environment
- Mypy version used: v0.960
- Mypy command-line flags: none
- Mypy configuration options from
mypy.ini(and other config files): none - Python version used: 3.10.4
- Operating system and version: Archlinux
Guide de contribution
Ouvrir 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.
Piste de recherche
Reproduisez le rapport avec l’exemple test.py fourni, puis examinez le commit 03901ef657daf6f477e6fe933c42ac270cf5034b et les modifications de dataclass-transform qu’il a introduites. Déterminez pourquoi l’annotation sur collections affecte l’annotation ultérieure et vérifiez que l’exemple ne produit plus d’erreur false-positive name-defined.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- devtools
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 42/100