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 typescript to builtin html-language-features #34

Merged
merged 1 commit into from
Jul 22, 2020
Merged

Conversation

marcdumais-work
Copy link
Contributor

@marcdumais-work marcdumais-work commented Jul 22, 2020

Fixes #33

Signed-off-by: Marc Dumais marc.dumais@ericsson.com

@marcdumais-work
Copy link
Contributor Author

Not yet tested

Fixes #33

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
@marcdumais-work
Copy link
Contributor Author

I had the chance to test this PR using Gitpod, comparing the behaviour vs master, with an HTML file containing embedded JS code. I think it now works! @akosyakov is on vacation this week, so I think I'll go ahead and merge.

I have confirmed that TypeScript and HTML support seem to work fine with the latest vscode baseline (from ~12h ago) so I expect that we'll get one working next/preview version of html-language-features after the nightly build, later today.

Unless we take special measures, only new releases (solid and nightly) will be covered by this fix - it would be more complicated to remove from open-vsx.org and re-publish previous versions that are already there.

cc: @spoenemann

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.

html-language-features should be built with typescript dependency
1 participant