false-positive `name-defined` regression with dataclasses and shadowing
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
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
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.
Línea de trabajo
Reproduce el informe con el ejemplo test.py proporcionado y, después, inspecciona el commit 03901ef657daf6f477e6fe933c42ac270cf5034b y los cambios de dataclass-transform que introdujo. Rastrea por qué la anotación sobre collections afecta a la anotación posterior y verifica que el ejemplo ya no produce un error false-positive name-defined.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- devtools
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 42/100