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

Fix tsserver not returning details for items with empty source #2845

Merged
merged 1 commit into from
Oct 18, 2019

Conversation

DonnieWest
Copy link
Contributor

Prior to #2709 tsserver based completions would include details on more items, whereas now there are fewer items with details.

This is caused by including an empty string as the source when communicating with the tsserver, which confuses it and forces it to return less information about completed items.

This resolves the issue, and fixes the requisite tests

This was referenced Oct 18, 2019
@w0rp w0rp merged commit 9125ec8 into dense-analysis:master Oct 18, 2019
@w0rp
Copy link
Member

w0rp commented Oct 18, 2019

Cheers! 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants