Should dict() constructor overloads be more permissive?
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 5.1k
- Forks
- 2.1k
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 82
Descripción
Current constructor overloads are here:
Someone took care to allow specific positive case, and to block specific negative case.
Many potential valid uses have fallen through the cracks:
dict([[1, 2]])
dict("AT TA GC CG".split())
dict([["str", b"bytes"]])
My understanding of the issue is that there's no way to specify sequence length (other than a tuple) in a type hint, that is there's no syntax to hint ["a", "b"] vs ["a", "b", "c"].
This brings a philosophical question: what side should typeshed err on when type hint syntax is not precise enough?
- type whatever Python may accept run time, or
- restrict users to what Python is guaranteed to accept at run time?
It was mentioned at https://github.com/microsoft/pyright/issues/7382 that type checkers trust typeshed, and thus the question belongs here.
My personal preference would be for permissive type hints in these cases.
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
Comienza con stdlib/builtins.pyi, en las sobrecargas enlazadas del constructor dict(), y compáralas con las llamadas de ejemplo del issue. Comprueba cómo los type checkers relevantes gestionan esos casos y determina después si la política prevista es aceptar entradas más amplias que sean válidas en tiempo de ejecución; para darlo por terminado se requieren un diseño de sobrecargas acordado y la validación correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 25/100