Wrong return type inferred for overload with a type variable
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Bug Report
from typing import TypeVar, overload
_T = TypeVar("_T")
@overload
def getenv(key: str) -> str | None: ...
@overload
def getenv(key: str, default: _T) -> str | _T: ...
def getenv(key, default):
return key
def test(custom_var: str) -> str:
return getenv(custom_var) or getenv("V", "D")
https://mypy-play.net/?mypy=latest&python=3.12&gist=4d959cd7069a8495533bfeca1cd40729
Actual Behavior
main.py:16: error: Incompatible return value type (got "str | None", expected "str") [return-value]
Expected Behavior
This should type check without errors, since we have an or with a call that always returns str.
It works if I replace type variable with just str:
@overload
def getenv(key: str, default: str) -> str: ...
but something about _T and the union upsets it.
Your Environment
- Mypy version used: 1.13.0
- Mypy command-line flags: none
- Mypy configuration options from
mypy.ini(and other config files): none - Python version used: 3.10-3.13
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der bereitgestellten Python-Reproduktion und bestätige das Verhalten mit mypy 1.13.0 ohne Konfigurationsflags. Verfolge die Auflösungen der Überladungen und die Inferenz der Typvariablen für die beiden getenv-Aufrufe; füge anschließend einen Regressionstest hinzu, der zeigt, dass der Ausdruck akzeptiert wird, wenn der Fallback str zurückgibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100