-
Notifications
You must be signed in to change notification settings - Fork 13.4k
'declare method' quick fix adds inline import #37781
Copy link
Copy link
Closed
Labels
BugA bug in TypeScriptA bug in TypeScriptDomain: LS: Auto-importDomain: LS: Quick FixesEditor-provided fixes, often called code actions.Editor-provided fixes, often called code actions.Effort: ModerateRequires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".Requires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".FixedA PR has been merged for this issueA PR has been merged for this issueHelp WantedYou can do thisYou can do this
Milestone
Metadata
Metadata
Assignees
Labels
BugA bug in TypeScriptA bug in TypeScriptDomain: LS: Auto-importDomain: LS: Quick FixesEditor-provided fixes, often called code actions.Editor-provided fixes, often called code actions.Effort: ModerateRequires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".Requires experience with the TypeScript codebase, but feasible. Harder than "Effort: Casual".FixedA PR has been merged for this issueA PR has been merged for this issueHelp WantedYou can do thisYou can do this
Type
Fields
Give feedbackNo fields configured for issues without a type.
From microsoft/vscode#94118
TypeScript Version: 3.9.0-dev.20200330
Search terms
Repro
For a project:
foo.tsindex.tsdeclare methodquick fix onbazExpected behavior:
An import for
Foois added to the top of the file:Actual behavior:
An inline import is added:
Playground Link:
Related Issues: