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 VS Code API compliance #8617

Merged
merged 2 commits into from
Oct 12, 2020
Merged

Fix VS Code API compliance #8617

merged 2 commits into from
Oct 12, 2020

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Oct 10, 2020

What it does

Upstream vscode has added new constraints on types: microsoft/vscode@876ea86

Make it compliant

Also update default VS Code API to the current VS Code version

How to test

Try out plug-ins like vscode-java etc using some providers, they should work as usual

Review checklist

Reminder for reviewers

@benoitf benoitf marked this pull request as draft October 11, 2020 07:17
related to this VS Code commit: microsoft/vscode@876ea86

Change-Id: Ic1f8d34dcfe0c5ad87e540517b4cb1d21a48d323
Signed-off-by: Florent Benoit <[email protected]>
Change-Id: I5ef6376431d2ec9ed519d5f2f6b46053542736cc
Signed-off-by: Florent Benoit <[email protected]>
@benoitf benoitf marked this pull request as ready for review October 11, 2020 08:04
@benoitf benoitf added the vscode issues related to VSCode compatibility label Oct 11, 2020
Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

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

I've checked autocompletion, tasks, workspace symbols. Everything works the same as from the master branch.

@benoitf benoitf merged commit e33708b into master Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants