Mypy doesn't detect new metaclass on subclass
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
If you subclass a class with a metaclass but change the metaclass to your own with a different method signature, mypy does not pick up on that.
To Reproduce
Example:
from enum import EnumType, StrEnum as _StrEnum, auto
class EnumMeta(EnumType):
def __call__(cls, value=None, **kwargs):
# Return first enum if value is None
if value is None:
value = next(iter(cls))
return super().__call__(value, **kwargs)
class StrEnum(_StrEnum, metaclass=EnumMeta):
@staticmethod
def _generate_next_value_(name, *args):
# auto() defaults to name.lower()
return name
class FingerprintSource(StrEnum):
can = auto()
fw = auto()
fixed = auto()
FingerprintSource()
Out: <FingerprintSource.can: 'can'>
Expected Behavior
Mypy passes
Actual Behavior
selfdrive/car/data_structures.py:310: error: Missing positional argument "value" in call to "FingerprintSource" [call-arg]
Your Environment
mypy 1.11.1 (compiled: yes)
Python 3.11.4
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
Empieza ejecutando el ejemplo de Python proporcionado con mypy 1.11.1 y confirma el falso error de argumento faltante. Después, sigue cómo mypy determina la firma invocable de FingerprintSource cuando su subclase declara EnumMeta; añade cobertura específica para este caso y verifica que el ejemplo pase la comprobación de tipos sin el error.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100