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

Align terminal command ids with VS Code. Fixes #12084 #12134

Merged

Conversation

tsmaeder
Copy link
Contributor

@tsmaeder tsmaeder commented Feb 1, 2023

What it does

This PR aligns the command ids of commands related to terminals with their counterparts in VS Code.
The change includes a "drive-by" fix for the file open dialog that I needed for testing "Open in Terminal".

Fixes #12084

Contributed on behalf of STMicroelectronics

How to test

The attached extension contributes commands that invokes each terminal command via its VS Code id.

Review checklist

Reminder for reviewers

Contributed on behalf of STMicroelectronics

Signed-off-by: Thomas Mäder <[email protected]>
@tsmaeder
Copy link
Contributor Author

tsmaeder commented Feb 1, 2023

Here's the extension:

terminalcomms-0.0.1.zip

and the source:

terminalcomms.zip

@tsmaeder
Copy link
Contributor Author

tsmaeder commented Feb 1, 2023

@vince-fugnitto the promised follow-up to #12066

@tsmaeder tsmaeder force-pushed the 12084_align_terminal_commands branch from 89b4bfa to 135e389 Compare February 1, 2023 16:38
@vince-fugnitto vince-fugnitto added terminal issues related to the terminal vscode issues related to VSCode compatibility labels Feb 2, 2023
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

LGTM 👍 I confirmed the terminal commands executed by the provided plugin work as expected.

packages/plugin-ext/src/plugin/known-commands.ts Outdated Show resolved Hide resolved
Signed-off-by: Thomas Mäder <[email protected]>
@tsmaeder tsmaeder merged commit ec6bf6c into eclipse-theia:master Feb 8, 2023
@vince-fugnitto vince-fugnitto added this to the 1.35.0 milestone Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
terminal issues related to the terminal vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Align Terminal Command Ids with VS Code
3 participants