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

vscode: add TerminalQuickFixProvider API stubbing #12532

Merged
merged 1 commit into from
May 18, 2023

Conversation

vince-fugnitto
Copy link
Member

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

  • replace the vscode.npm builtin with 1.77.0 (npm-1.77.0.zip)
  • start the application using theia as a workspace
  • confirm that there are no activation errors on startup
  • confirm that npm scripts in the explorer is correctly populated (functionality like running a script should work)

Review checklist

Reminder for reviewers

@vince-fugnitto vince-fugnitto added vscode issues related to VSCode compatibility builtins Issues related to VS Code builtin extensions labels May 15, 2023
@vince-fugnitto vince-fugnitto self-assigned this May 15, 2023
@vince-fugnitto vince-fugnitto force-pushed the vf/builtin-terminalQuickFixProvider branch from 30c9312 to 1346558 Compare May 15, 2023 18:53
Copy link
Contributor

@alvsan09 alvsan09 left a 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>
@vince-fugnitto vince-fugnitto force-pushed the vf/builtin-terminalQuickFixProvider branch from 1346558 to d90dd86 Compare May 18, 2023 14:27
Copy link
Contributor

@marcdumais-work marcdumais-work left a 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

Copy link
Contributor

@marcdumais-work marcdumais-work 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 Vince!

@vince-fugnitto vince-fugnitto merged commit 39dfc8c into master May 18, 2023
@vince-fugnitto vince-fugnitto deleted the vf/builtin-terminalQuickFixProvider branch May 18, 2023 14:49
@github-actions github-actions bot added this to the 1.38.0 milestone May 18, 2023
tsmaeder pushed a commit to tsmaeder/theia that referenced this pull request May 23, 2023
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builtins Issues related to VS Code builtin extensions vscode issues related to VSCode compatibility
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[proposed API][builtins] Theia misses proposed API: TerminalQuickFixProvider
3 participants