Use empty outer context when checking str.format() calls
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Merge medio
- 1 d 18 h
- PR fusionados (30 d)
- 54
Descripción
This is a follow up for https://github.com/python/mypy/pull/7418
Because mypy is still too eager about outer context and str.format() has object type for its arguments, passing a call to generic function as an argument to str.format() may cause false positives. For example:
'{:.3f}'.format(sum(list_of_floats))
Fails with
Incompatible types in string interpolation (expression has type "object", placeholder has type "Union[int, float]")
A more rectified test case:
[case testFormatCallGenerics]
from typing import TypeVar, Union
T = TypeVar('T')
def foo(x: T) -> Union[T, int]: ...
x: str
y: float
"{:.3f}".format(foo(x)) # E: Incompatible types in string interpolation (expression has type "str", placeholder has type "Union[int, float]")
"{:.3f}".format(foo(y))
[builtins fixtures/primitives.pyi]
A potential simple fix is to re-accept the expression in empty context before checking the format specifier type.
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 comportamiento del caso de prueba testFormatCallGenerics usando los ejemplos genéricos de foo y str.format. Sigue la comprobación de llamadas a str.format de mypy y su gestión del contexto externo; terminado significa que la llamada válida de formateo de float ya no produce un falso positivo, mientras que el error previsto del especificador de formato sigue cubierto.
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
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100