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 broken installation of TS definition files for Deno. #77

Merged
merged 1 commit into from
Jun 6, 2020

Conversation

denisbrodbeck
Copy link
Contributor

Fix wrong VS Code Extension ID.
Explicitly await (and catch) deno.generateDtsForDeno, so all definition files are in place, before the extension finishes activation phase.

fixes #74, #66, #62, #48

Fix wrong VS Code Extension ID.
Explicitly `await` (and catch) `deno.generateDtsForDeno`, so all definition files are in place, before the extension finishes activation phase.

fixes denoland#74, denoland#66, denoland#62, denoland#48
@denisbrodbeck
Copy link
Contributor Author

A new release would be nice, I think many users are struggling with broken TypeScript definitions (for the Deno namespace).

Copy link
Member

@ry ry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks!

I'll push a new release now.

@ry ry merged commit ff87426 into denoland:master Jun 6, 2020
@nayeemrmn
Copy link
Collaborator

fixes #74, #66, #62, #48

@ry The last three linked issues weren't closed.

@denisbrodbeck I guess you have to write:

fixes #74, fixes #66, fixes #62, fixes #48.

@David-Else
Copy link

David-Else commented Jun 7, 2020

@ry There was no update triggered for the plugin in VS Code. If you completely remove the old version and reinstall then this fix is applied. It might be to do with not updating the version number?

EDIT: The update also broke this extension for me, now it does nothing #80 (comment) . (It doesn't seem to work at all for many people in the #80 issue thread)

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.

Why vs code doesn't show types for Deno
4 participants