microsoft / microsoft/TypeScript
Type inference for `Function.call` return type on generic functions
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.4k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
type infer function call generic
⏯ Playground Link
Playground link with relevant code
💻 Code
function numberEcho(x: number): number {
return x;
}
function genericEcho<T>(x: T): T {
return x;
}
// This works
const resultNumber: number = numberEcho.call(this, 1);
// Type Error: Type 'unknown' is not assignable to type 'string'.(2322)
const resultString: string = genericEcho.call(this, 'hello');
🙁 Actual behavior
Type inference of genericEcho.call return type failed even when all the argument types are known at call site.
Of course explicitly hinting the return type would work but it's not good:
const resultString: string = genericEcho.call<typeof this, [string], string>(this, 'hello');
🙂 Expected behavior
I would expect type inference of generic function return type to work if all arguments types are known.
I think #40179 is related but the example I provided here is much more common since it exists in standard library.
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 dem verlinkten TypeScript Playground und vergleiche die inferierten Rückgabetypen für die direkten und generischen Function.call-Beispiele. Als abgeschlossen gilt die Aufgabe, wenn der generische Aufruf den Argumenttyp inferiert und das Ergebnis als String akzeptiert, wobei das bestehende Verhalten weiterhin durch eine geeignete Regression-Prüfung abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 30/100