microsoft / microsoft/TypeScript
Inlay parameter hints don't account for trailing required parameters
Offen
Dieses Issue hat noch niemand übernommen.
7.0 LS Migration
Bug
Domain: LS: Inlay Hints
Help Wanted
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
🔎 Search Terms
inlay hints parameters rest spread argument
🕗 Version & Regression Information
- This is the behavior in every version I tried
⏯ Playground Link
N/A
💻 Code
/// <reference path="fourslash.ts" />
//// function test(...rest: [first: number, ...middle: string[], last: string]) {}
//// test(10, 'a', 'b', 'c')
verify.baselineInlayHints(undefined, {
includeInlayParameterNameHints: "all",
includeInlayFunctionParameterTypeHints: true,
});
🙁 Actual behavior
function test(...rest: [first: number, ...middle: string[], last: string]) {}
test(/* first: */10, /* ...middle: */'a', /* last: */'b', 'c')
🙂 Expected behavior
function test(...rest: [first: number, ...middle: string[], last: string]) {}
test(/* first: */10, /* ...middle: */'a', 'b', /* last: */'c')
Additional information about the issue
No response
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 bereitgestellten fourslash-Test und dem Aufruf verify.baselineInlayHints, und verfolge dann die TypeScript-Verarbeitung von Inlay-Hints für Tupel-Restparameter mit nachfolgenden erforderlichen Elementen. Aktualisiere die Regressionstests, sodass die erwartete Baseline mit der gezeigten Ausgabe übereinstimmt, und führe die relevanten fourslash- oder Inlay-Hints-Tests aus, um zu bestätigen, dass das nachfolgende Argument korrekt beschriftet wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 48/100