warn when default values on overloads differ from implementation signature
Aperta
Nessuno ha ancora preso questa issue.
feature
topic-overloads
- Lingua principale
- Python
- Stelle
- 20.6k
- Fork
- 3.3k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
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
}
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia riproducendo l’esempio di overload e segui il controllo delle firme di overload di mypy. Determina se il comportamento accettato debba essere un avviso per i defaults che differiscono dall’implementazione oppure un errore per qualsiasi default negli overload. Il lavoro è completo quando la regola scelta viene applicata in modo coerente ed è coperta da regression tests.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100