-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
vscode: add TerminalQuickFixProvider
API stubbing
#12532
Conversation
30c9312
to
1346558
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I confirm that the activation issue occurs before this changed and it's fixed with this change 👍
I have verified that the 'npm' scripts are visible in the explorer and they can be executed 👍
The code LGTM !!
The commit adds the stubbing for the proposed `TerminalQuickFixProvider` API which was necessary for the `vscode.npm` builtin to successfully activate at version `v1.77.0`. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
1346558
to
d90dd86
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good. Will quickly test locally
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks Vince!
The commit adds the stubbing for the proposed `TerminalQuickFixProvider` API which was necessary for the `vscode.npm` builtin to successfully activate at version `v1.77.0`. Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
What it does
Closes: #12522.
The pull-request adds stubbing for the proposed
TerminalQuickFixProvider
VS Code API.The API was necessary in order for
vscode.npm@1.77.0
to successfully activate.How to test
vscode.npm
builtin with1.77.0
(npm-1.77.0.zip)theia
as a workspacenpm scripts
in the explorer is correctly populated (functionality like running a script should work)Review checklist
Reminder for reviewers