false-positive `name-defined` regression with dataclasses and shadowing
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
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
Guida per i contributori
Apri 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.
Direzione di ricerca
Riproduci il report con l’esempio test.py fornito, quindi esamina il commit 03901ef657daf6f477e6fe933c42ac270cf5034b e le modifiche di dataclass-transform che ha introdotto. Traccia il motivo per cui l’annotazione su collections influisce sull’annotazione successiva e verifica che l’esempio non produca più un errore false-positive name-defined.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- devtools
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 42/100