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 missing deprecated show overload #11649

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

EstFoisy
Copy link
Contributor

@EstFoisy EstFoisy commented Sep 8, 2022

What it does

Add an old deprecated overload signature for OutputChannel.show in theia.d.ts and update the implementation accordingly.

Closes #11627

How to test

  1. include the following extension in your app (outputshow-0.0.1.vsix.zip)
  2. run the command "Output - Show Deprecated"
  3. confirm that the channel is shown

Review checklist

Reminder for reviewers

Add an old deprecated overload signature for OutputChannel.show in
theia.d.ts and update the implementation accordingly.
@EstFoisy EstFoisy changed the title add missing depricated show overload add missing deprecated show overload Sep 8, 2022
@vince-fugnitto vince-fugnitto added vscode issues related to VSCode compatibility output issues related to the output labels Sep 8, 2022
@msujew msujew self-requested a review September 9, 2022 14:43
Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍 I can confirm that the newly-added (but deprecated) way of showing the output works as expected.

@vince-fugnitto vince-fugnitto merged commit 26f9d94 into eclipse-theia:master Sep 12, 2022
@vince-fugnitto vince-fugnitto added this to the 1.30.0 milestone Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
output issues related to the output vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vscode: add missing OutputChannel.show
3 participants