python / python/mypy

Running `get_dynamic_class_hook()` after `get_class_decorator_hook_2()`

Abierto
#12,864 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

feature
Lenguaje dominante
Python
Estrellas
20.6k
Forks
3.3k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

Feature

It would be useful to run get_dynamic_class_hook() from plugins after get_class_decorator_hook_2() to support dynamic classes that inspect existing dataclasses / attrs classes.

Pitch

We have an internal mypy plugin that makes "partial" classes, where all required attributes are optional. It does this by examining the __init__ of an attrs class and creating a new __init__ for the new class where all args are now optional. This works in mypy 0.942, occasionally crashes in 0.950, and doesn't work at all in 0.960 (128661ccbf4396ed1538421c481f1773c7490169) because the attrs __init__ is not present during get_dynamic_class_hook().

Is there some way to work around this?

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza por seguir los puntos de entrada del plugin get_dynamic_class_hook() y get_class_decorator_hook_2() y cómo el orden de los hooks expone los métodos init generados por attrs. El cambio estará terminado cuando los hooks de clases dinámicas puedan inspeccionar dataclasses o clases de attrs existentes después del procesamiento de decoradores, con cobertura de regresión para el escenario del plugin de partial-class descrito aquí.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
compilers
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.