warn when default values on overloads differ from implementation signature
Abierto
Nadie ha tomado este issue todavía.
feature
topic-overloads
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
from typing import overload
@overload
def foo(value1: str, value2: bool = False) -> str: # this default value is wrong/useless
...
@overload
def foo(value1: int, value2: bool = False) -> int: # this default value is wrong/useless
...
def foo(value1: str | int, value2: bool = True) -> str | int:
...
in fact, maybe even prohibit defaults on overload signatures entirely, like in typescript:
function foo(value1: number, value2: boolean = false): number //error: A parameter initializer is only allowed in a function or constructor implementation.
function foo(value1: string, value2: boolean = false): string //error: A parameter initializer is only allowed in a function or constructor implementation.
function foo(value1: number | string, value2: boolean = true): number | string {
return 1
}
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 reproduciendo el ejemplo de overload y sigue la comprobación de firmas de overload de mypy. Determina si el comportamiento aceptado debería ser una advertencia para los defaults que difieren de la implementación o un error para cualquier default en los overloads. Se considera terminado cuando la regla elegida se aplica de forma coherente y está cubierta por pruebas de regresión.
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
- 35/100