Mypy should ensure overloads are compatible with the implementation
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 20.6k
- Forks
- 3.3k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug Report
Mypy should ensure overloads are compatible with the implementation
To Reproduce
from typing import Union, overload
from typing_extensions import Literal
@overload
def f(*, flag: Literal[True]) -> float: ...
@overload
def f(*, flag: Literal[False] = False) -> str: ...
def f(*, flag: bool = False) -> Union[str, float]:
return 5
reveal_type(f(flag=True))
reveal_type(f(flag=False))
reveal_type(f())
Expected Behavior
an error along the lines of "signature 2 incompatible with implementation" because it cannot return a str when flag=False
Actual Behavior
$ mypy test.py
test.py:11:13: note: Revealed type is "builtins.float"
test.py:12:13: note: Revealed type is "builtins.str"
test.py:13:13: note: Revealed type is "builtins.str"
Your Environment
- Mypy version used:
$ mypy --version
mypy 0.910
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par exécuter l’exemple d’overload fourni avec mypy 0.910 et confirmez les types révélés ainsi que le diagnostic manquant. Suivez les vérifications de compatibilité de la signature d’implémentation ; le travail est terminé lorsque les overloads incompatibles, comme le cas false-flag renvoyant str, produisent une erreur sans rejeter les overloads compatibles.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- tooling
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100