Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add exported members of all project files in the global completion list #17851

Closed
wants to merge 36 commits into from

Conversation

minestarks
Copy link
Member

This is the rebased and updated version of #13921.

  • Exported members of all project files are included in the global completion list, regardless of whether that file was already imported by the current file.

  • For completion list items coming from a not-yet-imported file, the returned CompletionEntry now has a new property hasAction, indicating if the user commits it, additional code actions will be needed to avoid errors.

  • To retrieve the related code actions, the editor should send a CompletionDetails request and read the new codeActions property.

Fixes #7849

…etSymbolsFromOtherSourceFileExports inside getCompletionData
…ntriesFromSymbols and to return value of getCompletionData
… symbolToOriginInfoMap (meaning there's an import action)
…eActionForImport, update call sites, destructure it, use compilerOptions in getModuleSpecifierForNewImport
@mhegazy
Copy link
Contributor

mhegazy commented Oct 17, 2017

Superseded by #19069

@mhegazy mhegazy closed this Oct 17, 2017
@microsoft microsoft locked and limited conversation to collaborators Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants